Skip to content

enh: allow con-duct pp usage.json#341

Closed
asmacdo wants to merge 1 commit intocon:mainfrom
asmacdo:pp-usage-file
Closed

enh: allow con-duct pp usage.json#341
asmacdo wants to merge 1 commit intocon:mainfrom
asmacdo:pp-usage-file

Conversation

@asmacdo
Copy link
Member

@asmacdo asmacdo commented Dec 5, 2025

Fixes #339

Draft because I think we are on the cusp of merging a large refactor #327

I'd rather fixup this one after that merges than delay that one

  • add test

@codecov
Copy link

codecov bot commented Dec 5, 2025

Codecov Report

❌ Patch coverage is 50.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 91.15%. Comparing base (e30a56c) to head (fdfed65).
⚠️ Report is 30 commits behind head on main.

Files with missing lines Patch % Lines
src/con_duct/suite/pprint_json.py 50.00% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #341      +/-   ##
==========================================
- Coverage   91.34%   91.15%   -0.19%     
==========================================
  Files           7        7              
  Lines         947      950       +3     
  Branches      125      126       +1     
==========================================
+ Hits          865      866       +1     
- Misses         59       60       +1     
- Partials       23       24       +1     

☔ View full report in Codecov by Sentry.
📢 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.

@asmacdo
Copy link
Member Author

asmacdo commented Dec 8, 2025

Closing in favor of

@asmacdo asmacdo closed this Dec 8, 2025
@asmacdo asmacdo deleted the pp-usage-file branch December 8, 2025 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug with pretty print

1 participant