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

update cicd with new commenting structure #3813

Open
wants to merge 21 commits into
base: v2.x/staging
Choose a base branch
from

Conversation

MarkAckert
Copy link
Member

This PR streamlines comments from github-actions[bot] into two PR comments which will be re-used and updated with the latest build and test status for actions run against said PR. Previously, the bot creates new comments on every build, which runs against every commit, creating a massive amount of comment noise on our PRs (200+).

We lose some build history in the PR thread with this change, but said history is typically not useful and can be recovered easily by looking at workflow history filtered against a branch.

This change also keeps the github-actions[bot] PR comments at the top of a PR, which might be undesirable in some cases, though I don't know of any cases where that's true yet. To address this, another workflow is created which can remove the github-actions[bot] comments from a PR. Run that manually, then the next build creates new comments at the bottom of the pull request.

@MarkAckert MarkAckert added the Test: None Used in CI label May 1, 2024
MarkAckert added 5 commits May 1, 2024 12:28
Signed-off-by: MarkAckert <[email protected]>
Signed-off-by: MarkAckert <[email protected]>
Signed-off-by: MarkAckert <[email protected]>
@MarkAckert MarkAckert removed the Test: None Used in CI label May 1, 2024
@zowe zowe deleted a comment from github-actions bot May 1, 2024
Signed-off-by: MarkAckert <[email protected]>
@zowe zowe deleted a comment from github-actions bot Aug 28, 2024
@zowe zowe deleted a comment from github-actions bot Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants