Skip to content

ci: use job summary instead of pr comment#7569

Merged
nbolton merged 1 commit intomasterfrom
job-summary
Sep 23, 2024
Merged

ci: use job summary instead of pr comment#7569
nbolton merged 1 commit intomasterfrom
job-summary

Conversation

@nbolton
Copy link
Copy Markdown
Member

@nbolton nbolton commented Sep 22, 2024

Fixes: #7568

The sea of PR comments is super annoying and noisy.

This PR fixes that by moving all summaries (tests, lint, analysis) to the workflow job summaries, then if there's anything interesting that needs action, a single, simple PR comment is added to show the PR author where to go for more info.

image

@nbolton nbolton force-pushed the job-summary branch 7 times, most recently from 77b67f1 to b401d97 Compare September 22, 2024 14:50
@deskflow deskflow deleted a comment from github-actions bot Sep 22, 2024
@nbolton nbolton force-pushed the job-summary branch 20 times, most recently from 7f6baf3 to 3d177b5 Compare September 23, 2024 00:29
@nbolton nbolton force-pushed the job-summary branch 7 times, most recently from 8c6ec42 to b1ec26a Compare September 23, 2024 02:10
@nbolton nbolton marked this pull request as ready for review September 23, 2024 02:10
@nbolton nbolton enabled auto-merge (rebase) September 23, 2024 02:14
@nbolton nbolton requested a review from sithlord48 September 23, 2024 02:15
@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Member

@sithlord48 sithlord48 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@nbolton nbolton merged commit 806ef46 into master Sep 23, 2024
@nbolton nbolton deleted the job-summary branch September 23, 2024 16:12
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.

Use GitHub step/job summary instead of PR comments

2 participants