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

cabal-validate: Output is too verbose and not verbose enough #10569

Open
9999years opened this issue Nov 20, 2024 · 1 comment · May be fixed by #10573
Open

cabal-validate: Output is too verbose and not verbose enough #10569

9999years opened this issue Nov 20, 2024 · 1 comment · May be fixed by #10573
Labels
cabal-validate validate.sh (cabal-validate) test suite runner meta: tracking re: devx Improving the cabal developer experience (internal issue) type: testing Issues about project test suites

Comments

@9999years
Copy link
Collaborator

./validate.sh starts by printing 93 lines of output (runtime configuration, tool and compiler versions, the names and versions of all transitive dependencies and local packages) before anything gets built. This is regardless of whether or not --verbose is passed.

Then, unless you pass --verbose, it hides the cabal build output! A clean build can take several minutes, so it's annoying to have no indication of how close it is to being done or what it's doing.

Then, regardless of if --verbose is passed, test suites won't output any indication that they've run a test! This is because the default Tasty arguments include --hide-successes, there's no --no-hide-successes option, and ./validate.sh --tasty-arg ... can only add arguments, not replace the default ones.

Proposal:

  • print-config, print-tool-versions, and the full build plan output (currently inseparable from the build step) should be hidden by default, unless --verbose is given.
    • If we want, we could write this information to a file somewhere to preserve it.
  • cabal build output should be shown by default. Maybe a --quiet option could be added to hide it?
  • --verbose should toggle --hide-successes for test suites.
@9999years 9999years added meta: tracking re: devx Improving the cabal developer experience (internal issue) type: testing Issues about project test suites cabal-validate validate.sh (cabal-validate) test suite runner labels Nov 20, 2024
@geekosaur
Copy link
Collaborator

Small suggestion: if stdout isn't a tty, maybe output GitHub Actions grouping that can be collapsed or expanded by the viewer instead of the ANSI escapes. Then we can make everything verbose in CI without overwhelming people trying to go through the log.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cabal-validate validate.sh (cabal-validate) test suite runner meta: tracking re: devx Improving the cabal developer experience (internal issue) type: testing Issues about project test suites
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants