Keep structured command output clean - #1831
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1831 +/- ##
=======================================
Coverage 65.97% 65.97%
=======================================
Files 43 43
Lines 8434 8436 +2
=======================================
+ Hits 5564 5566 +2
Misses 2870 2870
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
can you explain why, this isnt traditional for a cli, in most cases youd want something human readable in stdout unless the user gives a --json flag for only json output |
|
Updated based on this feedback: the affected commands now have an explicit |
Summary
--jsonto file conversion, project publication, and project upload--json,--format=json, and--format=yamlby sending status messages to stderr--jsonwith--formatTests
cargo fmt --all -- --checkcargo check --all-targetscargo clippy --all-targets -- -D warningscargo test cmd_project::testscargo test cmd_file::test--json/--formatconflictThe token-backed conversion integration test now covers both the unchanged default output and the new
--jsonbehavior in CI.Closes #1757