exercism / jq

Exercism exercises in jq.
https://exercism.org/tracks/jq
MIT License
8 stars 31 forks source link

Enable debug output #244

Closed glennj closed 4 months ago

glennj commented 4 months ago

The purpose of this PR is to separate jq's debug output from the rest of the stderr output, so that bats can handle it properly.

This is a companion PR to exercism/jq-test-runner#33 -- I suggest merging that one first.

This is a pretty big PR. You might want to read each commit individually.

github-actions[bot] commented 4 months ago

This PR touches files which potentially affect the outcome of the tests of an exercise. This will cause all students' solutions to affected exercises to be re-tested.

If this PR does not affect the result of the test (or, for example, adds an edge case that is not worth rerunning all tests for), please add the following to the merge-commit message which will stops student's tests from re-running. Please copy-paste to avoid typos.

[no important files changed]

For more information, refer to the documentation. If you are unsure whether to add the message or not, please ping @exercism/maintainers-admin in a comment. Thank you!

ErikSchierboom commented 4 months ago

You'll likely want to use the no important files changed marker

glennj commented 4 months ago

Updated the exercises/shared/.docs/{help,tests}.md files -- these are combined to create the HELP.md file downloaded by the CLI.

barabo commented 4 months ago

I love this! Thanks @glennj for putting this together!

glennj commented 4 months ago

@IsaacG when you get a chance, some changes for review.