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

Add 'minimal' output format #431

Merged
merged 7 commits into from
Aug 2, 2022
Merged

Add 'minimal' output format #431

merged 7 commits into from
Aug 2, 2022

Conversation

foresterre
Copy link
Owner

@foresterre foresterre commented Aug 1, 2022

This output format only reports the final result of a subcommand.
If the subcommand fails with an unexpected error, no output may be given,
but a failure exit code can still be expected.

  • Changelog
  • Book
  • Add VerifyOutput to verify
  • Consider printing oneline errors Monitoring: if this is desired: please open an issue

closes #370

This output format only reports the final result of a subcommand.
If the subcommand fails with an unexpected error, no output may be given,
but a failure exit code can still be expected.
@codecov
Copy link

codecov bot commented Aug 1, 2022

Codecov Report

Merging #431 (d3aecbc) into main (49140b3) will increase coverage by 0.65%.
The diff coverage is 58.00%.

@@            Coverage Diff             @@
##             main     #431      +/-   ##
==========================================
+ Coverage   59.86%   60.51%   +0.65%     
==========================================
  Files          76       79       +3     
  Lines        1744     1811      +67     
==========================================
+ Hits         1044     1096      +52     
- Misses        700      715      +15     
Impacted Files Coverage Δ
src/bin/cargo-msrv.rs 0.00% <0.00%> (ø)
src/config.rs 68.13% <0.00%> (+3.21%) ⬆️
src/lib.rs 35.29% <ø> (ø)
src/reporter.rs 61.53% <ø> (ø)
src/reporter/event.rs 50.00% <ø> (ø)
src/reporter/handler/human_progress_handler.rs 0.00% <0.00%> (ø)
src/reporter/handler/json_handler.rs 0.00% <ø> (ø)
src/sub_command/verify.rs 70.21% <47.05%> (-9.79%) ⬇️
src/reporter/event/shared/compatibility.rs 57.14% <57.14%> (ø)
src/reporter/handler/minimal_output_handler.rs 65.71% <65.71%> (ø)
... and 10 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2004f94...d3aecbc. Read the comment docs.

…result

* Renamed outer "Compatibility" struct to "CheckResult"
* Moved inner "Compatibility" contents of struct to a shared module, named still as "Compatibility"
* Renamed "CompatibilityCheckMethod" to "CheckMethod" for consistency
While 'verify' already kind of outputs its final result by issuing a CheckResult event, and by virtue of it consisting of a single check run, we should also output a final result, even if just for consistency.
@foresterre
Copy link
Owner Author

bors r+

@bors bors bot merged commit 4b2e329 into main Aug 2, 2022
@foresterre foresterre deleted the minimal branch August 2, 2022 22:56
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.

Support simple output format
1 participant