Skip to content

Add --split-files/--no-split-files CLI options#83

Merged
tsalo merged 3 commits into
mainfrom
fix/enable_split_output
Jul 1, 2026
Merged

Add --split-files/--no-split-files CLI options#83
tsalo merged 3 commits into
mainfrom
fix/enable_split_output

Conversation

@araikes

@araikes araikes commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator

Add explicit CLI options to control output splitting behavior:

  • --split-files: writes one output per scalar
  • --no-split-files: combines all scalars to one output

Smart defaults are applied when omitted:

  • Wide cohorts (with --scalar-columns) split by default
  • Long-format cohorts combine by default

Updates documentation with new "Output splitting" section and consolidates repeated explanations across command sections. Includes comprehensive tests for the routing logic and CLI argument parsing.

Closes #81

Add explicit CLI options to control output splitting behavior:
- --split-files: writes one output per scalar
- --no-split-files: combines all scalars to one output

Smart defaults are applied when omitted:
- Wide cohorts (with --scalar-columns) split by default
- Long-format cohorts combine by default

Updates documentation with new "Output splitting" section and consolidates repeated explanations across command sections. Includes comprehensive tests for the routing logic and CLI argument parsing.
@araikes araikes requested a review from tsalo June 30, 2026 16:50
@codecov-commenter

codecov-commenter commented Jun 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.81%. Comparing base (89226a8) to head (9a9f889).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #83      +/-   ##
==========================================
+ Coverage   90.77%   90.81%   +0.03%     
==========================================
  Files          21       21              
  Lines        1463     1469       +6     
  Branches      304      305       +1     
==========================================
+ Hits         1328     1334       +6     
  Misses         76       76              
  Partials       59       59              

☔ 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.

@tsalo tsalo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks for this!

@tsalo tsalo merged commit c435f0c into main Jul 1, 2026
8 checks passed
@tsalo tsalo deleted the fix/enable_split_output branch July 1, 2026 17:11
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.

Expose split-output behavior as a configurable CLI option

3 participants