Skip to content

Reject unsupported KCL format output - #1825

Merged
JordanNoone merged 1 commit into
mainfrom
fix-1751-reject-unsupported-format
Aug 30, 2026
Merged

Reject unsupported KCL format output#1825
JordanNoone merged 1 commit into
mainfrom
fix-1751-reject-unsupported-format

Conversation

@JordanNoone

Copy link
Copy Markdown
Contributor

Summary

  • reject YAML and table output explicitly for zoo kcl format
  • avoid successful commands that silently produce empty stdout
  • keep JSON and default formatted-source output unchanged

Tests

  • cargo fmt --check
  • ZOO_TEST_TOKEN=dummy cargo test format_kcl_rejects_unsupported_output --no-fail-fast

Closes #1751

@codecov

codecov Bot commented Aug 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.97%. Comparing base (0c27251) to head (20f718b).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1825      +/-   ##
==========================================
+ Coverage   65.94%   65.97%   +0.02%     
==========================================
  Files          43       43              
  Lines        8434     8434              
==========================================
+ Hits         5562     5564       +2     
+ Misses       2872     2870       -2     
Flag Coverage Δ
unittests 65.97% <ø> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@JordanNoone
JordanNoone marked this pull request as ready for review August 29, 2026 23:45
@JordanNoone
JordanNoone enabled auto-merge (squash) August 29, 2026 23:45
@JordanNoone
JordanNoone merged commit d944e7d into main Aug 30, 2026
30 of 31 checks passed
@JordanNoone
JordanNoone deleted the fix-1751-reject-unsupported-format branch August 30, 2026 14:09
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.

zoo kcl format --format yaml|table exits 0 with empty stdout

2 participants