-
-
Notifications
You must be signed in to change notification settings - Fork 739
fix: Update CI workflow versions to latest #713
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
fix: Update CI workflow versions to latest #713
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This and all other PRs look good to me, but they are marked as drafts. Feel free to merge them, or assign me as a reviewer if you want me to take a look at them.
|
@antonbabenko theres a few repos that I'll need to make some manual tweaks but wanted to get your thoughts on the changes before proceeding. Specifically, moving the changelog and the config for the automatic release generation to |
|
and FYI - I updated the release workflow and validated here terraform-aws-modules/terraform-aws-eks@master...clowdhaus:terraform-aws-eks:master to ensure its working as intended |
|
I like that the resulting CHANGELOG, as well as other config files (pre-commit, releaserc), is in the root. This is the default location for such things. I also like that "docs" folder contains more specific docs (by human) and CHANGELOG (by script) is in the root. I would not change it. |
## [8.1.1](v8.1.0...v8.1.1) (2025-10-21) ### Bug Fixes * Update CI workflow versions to latest ([#713](#713)) ([feb4561](feb4561))
|
This PR is included in version 8.1.1 🎉 |
Description
.gitignorewith other modulesrmzto speed up unnecessary file removal in GitHub CI environment. There is now a conditional for thepre-commit run -astep (namedMax TF pre-commitin the GitHub UI) that will delete more files for the security group module since it will consume a lot of space due to all the sub-modules. Otherwise we remove just enough in the quickest fashion to get on with business.Motivation and Context
Breaking Changes
How Has This Been Tested?
examples/*to demonstrate and validate my change(s)examples/*projectspre-commit run -aon my pull request