Skip to content

Replace specific Terraform setup actions with wildcard#682

Closed
DaanHoogland wants to merge 1 commit intoapache:mainfrom
DaanHoogland:patch-1
Closed

Replace specific Terraform setup actions with wildcard#682
DaanHoogland wants to merge 1 commit intoapache:mainfrom
DaanHoogland:patch-1

Conversation

@DaanHoogland
Copy link
Copy Markdown

No description provided.

Signed-off-by: dahn <daan.hoogland@gmail.com>
Copy link
Copy Markdown
Member

@raboof raboof left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As noted at the top of this file, please don't edit it manually - the 'source of truth' is actions.yaml.

However, even there, we are migrating away from wildcard imports, and instead requiring explicitly pinned actions #666

Is there anything in particular that makes specifically-reviewed setup-terraform versions difficult?

@DaanHoogland
Copy link
Copy Markdown
Author

@raboof , the already configured versions don’t seem to pass.

@raboof
Copy link
Copy Markdown
Member

raboof commented Apr 8, 2026

@raboof , the already configured versions don’t seem to pass.

Can you point at a specific instance of the problem?

@DaanHoogland
Copy link
Copy Markdown
Author

DaanHoogland commented Apr 8, 2026

@raboof , the already configured versions don’t seem to pass.

Can you point at a specific instance of the problem?

ja, picnic. I/we was using a version instead of a hash, still

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants