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

Filter logs #1050

Merged
merged 4 commits into from
Nov 1, 2023
Merged

Filter logs #1050

merged 4 commits into from
Nov 1, 2023

Conversation

mih
Copy link
Collaborator

@mih mih commented Nov 1, 2023

Closes #1048

mih and others added 4 commits November 1, 2023 13:05
These consume a lot of space, and are often not helpful given the simple
usage patterns that dominate the handbook.

This change removes any action summary that does not contain notneeded.
The exception is useful, because by default notneeded results are not
renderer individually.

Ping #1009
Ping #1048
Comments are inside the diff
Copy link
Contributor

@adswa adswa left a comment

Choose a reason for hiding this comment

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

much better! One of the info messages "[INFO] Attempting a clone into" still isn't caught, though...

@adswa
Copy link
Contributor

adswa commented Nov 1, 2023

(merging anyway already, to see the impact on length)

@adswa adswa merged commit 464ee94 into main Nov 1, 2023
2 checks passed
@adswa adswa deleted the filter-logs branch November 1, 2023 14:58
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.

Remove needless INFO messages and action summary output
2 participants