Collapse/hide environment deployments inside PRs #67649
Unanswered
artis3n
asked this question in
Pull Requests
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Select Topic Area
Product Feedback
Body
It would be excellent for there to be an option to collapse environment deployment notices in PRs, or move them to a dedicated tab with only the most recent present in the Conversations tab, because leveraging environments with many automated tests makes it impossible to communicate on the PR. For example, I am currently collaborating with a peer in digitalocean/ansible-collection#45, which currently runs 52 tests. To protect secrets, the tests are wrapped under a GitHub environment with a manual approval gate. This means that every time a commit is pushed, 51 notices appear on the PR about each deployment to the "integration" environment. The last PR review with comments I submitted to this PR is completely hidden from the author of the PR.
And, expanding that view to find the bottom of the PR where there's a review comment to respond to is ridiculous, because it expands from the top down.
This specific situation may be somewhat niche, but the most common pattern I've seen to safely allow external contributions to open source repos where secrets are needed to run tests in PRs is to gate execution of the workflows behind a GitHub environment with an approval step, and therefore anyone following that safe pattern is going to hit this frustrating UX.
There is a "comments" section of the Files Changed view to find comments left on files in the PR, but review comments -
are gone from the Conversations tab, and hidden under too many deployment notifications to find. Help :(
Beta Was this translation helpful? Give feedback.
All reactions