Skip to content

ci: authenticate tflint init - #5894

Merged
ferrarimarco merged 1 commit into
mainfrom
auth-tflint-init
Jul 16, 2024
Merged

ferrarimarco merged 1 commit into
mainfrom
auth-tflint-init

Conversation

@ferrarimarco

Copy link
Copy Markdown
Collaborator

Proposed changes

TFLint init might hit an API rate limit, especially when requests to the TFLint backend come from shared tenants, such as GitHub Actions.

Readiness checklist

In order to have this pull request merged, complete the following tasks.

Pull request author tasks

  • I checked that all workflows return a success.
  • I included all the needed documentation for this change.
  • I provided the necessary tests.
  • I squashed all the commits into a single commit.
  • I followed the Conventional Commit v1.0.0 spec.
  • I wrote the necessary upgrade instructions in the upgrade guide.
  • If this pull request is about and existing issue,
    I added the Fix #ISSUE_NUMBER or Close #ISSUE_NUMBER text to the description of the pull request.

Super-linter maintainer tasks

  • Label as breaking if this change breaks compatibility with the previous released version.
  • Label as either: automation, bug, documentation, enhancement, infrastructure.
  • Add the pull request to a milestone, eventually creating one, that matches with the version that release-please proposes.

TFLint init might hit an API rate limit, especially when requests to
the TFLint backend come from shared tenants, such as GitHub Actions.
@ferrarimarco ferrarimarco added infrastructure related to project structure and maintenance O: backlog 🤖 Backlog, stale ignores this label labels Jul 16, 2024
@ferrarimarco ferrarimarco added this to the 6.8.0 milestone Jul 16, 2024
@ferrarimarco ferrarimarco self-assigned this Jul 16, 2024
@ferrarimarco
ferrarimarco merged commit cc20e45 into main Jul 16, 2024
@ferrarimarco
ferrarimarco deleted the auth-tflint-init branch July 16, 2024 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infrastructure related to project structure and maintenance O: backlog 🤖 Backlog, stale ignores this label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant