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

feat: add env var to silence pr comments #4415

Closed
wants to merge 4 commits into from

Conversation

anryko
Copy link
Contributor

@anryko anryko commented Apr 10, 2024

Silence PR Comments

PoC implementation for the simplest case of silencing PR comments via setting ATLANTIS_SILENCE_PR_COMMENTS=true (default false) environment variable.

ℹ️ This is not a complete feature. Just testing if this approach is acceptable before taking on the rest of the functionality.

why

When a user has many Atlantis servers planning Terraform change it causes a lot of comment noise in the GitHub PR thread. They might want to rely solely on the GitHub status checks in those cases.

tests

  • I have tested my changes by running a local development instance and manually testing the workflow.

references

@anryko anryko requested review from a team as code owners April 10, 2024 11:46
@anryko anryko requested review from jamengual, nitrocode and X-Guardian and removed request for a team April 10, 2024 11:46
@github-actions github-actions bot added the go Pull requests that update Go code label Apr 10, 2024
@jamengual jamengual added needs discussion Large change that needs review from community/maintainers waiting-on-review Waiting for a review from a maintainer labels Apr 10, 2024
@anryko
Copy link
Contributor Author

anryko commented May 13, 2024

Closing in favor of #4543.

@anryko anryko closed this May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
go Pull requests that update Go code needs discussion Large change that needs review from community/maintainers waiting-on-review Waiting for a review from a maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add an option to disable github plan/apply/policy_check comments for a workflow
2 participants