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

Jump to bottom

Update GitHub Workflows to Fix ReviewDog TFLint Action #56

Merged
merged 1 commit into from
May 7, 2024
Merged

Update GitHub Workflows to Fix ReviewDog TFLint Action #56

merged 1 commit into from
May 7, 2024

Conversation

osterman
Copy link
Member

@osterman osterman commented May 7, 2024

what

  • Update workflows (.github/workflows) to add issue: write permission needed by ReviewDog tflint action

why

  • The ReviewDog action will comment with line-level suggestions based on linting failures

@osterman osterman requested a review from a team as a code owner May 7, 2024 23:12
@osterman osterman added the auto-update This PR was automatically generated label May 7, 2024
@osterman osterman requested a review from a team as a code owner May 7, 2024 23:12
@osterman osterman added no-release Do not create a new release (wait for additional code changes) migration This PR involves a migration labels May 7, 2024
@osterman osterman merged commit c57b4d8 into main May 7, 2024
9 of 20 checks passed
@osterman osterman deleted the migration/20240507 branch May 7, 2024 23:12
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@johncblandii johncblandii Awaiting requested review from johncblandii johncblandii is a code owner automatically assigned from cloudposse/contributors

@nitrocode nitrocode Awaiting requested review from nitrocode nitrocode is a code owner automatically assigned from cloudposse/contributors

Assignees
No one assigned
Labels
auto-update This PR was automatically generated migration This PR involves a migration no-release Do not create a new release (wait for additional code changes)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@osterman