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

Support --quiet in CI script #12318

Open
mweatherley opened this issue Mar 5, 2024 · 0 comments · May be fixed by #13308
Open

Support --quiet in CI script #12318

mweatherley opened this issue Mar 5, 2024 · 0 comments · May be fixed by #13308
Labels
A-Build-System Related to build systems or continuous integration C-Enhancement A new feature

Comments

@mweatherley
Copy link
Contributor

mweatherley commented Mar 5, 2024

Follow-up from #12242

What problem does this solve or what need does it fill?

The CI test script produces a very large volume of output (especially with --keep-going), and almost all of it is not useful: it's just tests succeeding. With automated testing this isn't really an issue (especially since, at the moment, the test scripts crash shortly after anything fails), but when running tests locally, it can be annoying.

What solution would you like?

The ability to pass a --quiet flag, e.g. cargo run -p ci -- --quiet, cargo run -p ci -- --keep-going --quiet etc. could ameliorate this considerably. The flag would mostly mean passing --quiet to the constituent tests of the script so that it suppresses the flood of success outputs.

What alternative(s) have you considered?

Manually redirecting stdout to /dev/null does an okayish job of suppressing some of the output; e.g.: cargo run -p ci -- --keepgoing > /dev/null. However, this kills all of the output from our lints, and it doesn't suppress as much as --quiet would, so it's a very imperfect solution.

@mweatherley mweatherley added C-Enhancement A new feature S-Needs-Triage This issue needs to be labelled labels Mar 5, 2024
@alice-i-cecile alice-i-cecile added A-Build-System Related to build systems or continuous integration and removed S-Needs-Triage This issue needs to be labelled labels Mar 5, 2024
@moonlightaria moonlightaria linked a pull request May 10, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Build-System Related to build systems or continuous integration C-Enhancement A new feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants