Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add workflow to prevent additions to the Terraform SDK folder #842

Merged
merged 12 commits into from
Dec 12, 2024

Conversation

mjhilton
Copy link
Contributor

Fairly self-explanatory. Detects additions only, allows deletions and modifications.

Example successful run:
image

Example failed run:
image
image

@mjhilton mjhilton enabled auto-merge (squash) December 12, 2024 23:04
@denys-octopus
Copy link
Contributor

Nice 👍 Looks good. Sorry would this action failing prevent branch from merging?

@mjhilton mjhilton merged commit da3bf7f into main Dec 12, 2024
24 checks passed
@mjhilton mjhilton deleted the matth/prevent-new-sdk-additions branch December 12, 2024 23:20
@mjhilton
Copy link
Contributor Author

Nice 👍 Looks good. Sorry would this action failing prevent branch from merging?

Not 100% sure how GHA works in this sense. I don't think so, I think you need to specifically mark a workflow as "required" in GH before it'll block merges.

@mjhilton
Copy link
Contributor Author

Yep. We can choose to configure this as a blocking workflow if we want:
image

@mjhilton
Copy link
Contributor Author

Might keep it optional for a few PRs to ensure it works well, and think about adding it as mandatory in 2025.

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