Skip to content
New issue

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

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

Already on GitHub? # to your account

fix: Use a version for to avoid GitHub API rate limiting on CI workflows #73

Merged

Conversation

bryantbiggs
Copy link
Member

Description

  • Update pre-commit workflow CI checks to use pinned version for tflint to avoid GitHub API rate limiting and use latest action versions
  • Update pre-commit versions to latest

Motivation and Context

  • Avoid failing CI checks due to GitHub API rate limiting

Breaking Changes

  • No

How Has This Been Tested?

  • I have updated at least one of the examples/* to demonstrate and validate my change(s)
  • I have tested and validated these changes using one or more of the provided examples/* projects
  • I have executed pre-commit run -a on my pull request

@antonbabenko antonbabenko merged commit fbff232 into terraform-aws-modules:master Jan 3, 2023
@bryantbiggs bryantbiggs deleted the chore/update-ci branch January 3, 2023 14:23
antonbabenko pushed a commit that referenced this pull request Jan 24, 2023
### [4.1.3](v4.1.2...v4.1.3) (2023-01-24)

### Bug Fixes

* Use a version for  to avoid GitHub API rate limiting on CI workflows ([#73](#73)) ([fbff232](fbff232))
@antonbabenko
Copy link
Member

This PR is included in version 4.1.3 🎉

@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 24, 2023
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants