Skip to content

Conversation

@bearomorphism
Copy link
Collaborator

@bearomorphism bearomorphism commented Jan 15, 2026

I just remembered that we already have a feature that detects if a configuration file is valid. We can include pyproject.toml in the multi config warning.

Relate #1702

Also added a test to ensure that no warning when there are both pyproject.toml (without cz settings section) and cz.toml

@codecov
Copy link

codecov bot commented Jan 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.95%. Comparing base (cdaa941) to head (5eb1e1c).
⚠️ Report is 1 commits behind head on master.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1803      +/-   ##
==========================================
- Coverage   97.96%   97.95%   -0.01%     
==========================================
  Files          60       60              
  Lines        2648     2643       -5     
==========================================
- Hits         2594     2589       -5     
  Misses         54       54              

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

@bearomorphism bearomorphism force-pushed the multi-config-include-pyproject branch from 7514e8e to ec46a4c Compare January 15, 2026 16:39
@bearomorphism bearomorphism marked this pull request as draft January 15, 2026 17:17
@bearomorphism bearomorphism force-pushed the multi-config-include-pyproject branch 2 times, most recently from 4768276 to da1e65c Compare January 15, 2026 17:23
@bearomorphism bearomorphism marked this pull request as ready for review January 15, 2026 17:23
@bearomorphism bearomorphism force-pushed the multi-config-include-pyproject branch from da1e65c to ff90571 Compare January 15, 2026 17:55
@bearomorphism bearomorphism force-pushed the multi-config-include-pyproject branch from ff90571 to 5eb1e1c Compare January 15, 2026 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants