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

Deduplicate github action logs #1796

Merged
merged 11 commits into from
May 25, 2024
Merged

Conversation

redmoogle
Copy link
Contributor

@redmoogle redmoogle commented May 18, 2024

Deduplicate the logs

Fixes: #1778

only prints off the warnings from the ubuntu-latest run.

This is very slightly hacky but only matters for 1 warning (CA1416)

@github-actions github-actions bot added size/M and removed size/S labels May 18, 2024
@redmoogle
Copy link
Contributor Author

redmoogle commented May 18, 2024

will do some testing on my fork later 9actions are alot more fucky than I thought)

@redmoogle redmoogle closed this May 18, 2024
@redmoogle redmoogle reopened this May 19, 2024
@github-actions github-actions bot added size/S and removed size/M labels May 19, 2024
@github-actions github-actions bot added size/XS and removed size/S labels May 19, 2024
.github/workflows/build-test.yml Outdated Show resolved Hide resolved
@wixoaGit
Copy link
Member

Another possible approach could be to use the ::remove-matcher workflow command according to this documentation. I think how it would be used here would be to do echo "::remove-matcher owner=csc::" on non-ubuntu runners.

Co-authored-by: wixoa <wixoag@gmail.com>
@wixoaGit wixoaGit merged commit 2fe948a into OpenDreamProject:master May 25, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Build warnings/errors are displayed up to 3x in PR annotations
2 participants