Skip to content

more options for config file generation#1314

Merged
phlptp merged 13 commits into
CLIUtils:mainfrom
phlptp:defaults_for_inactive_subcommands
Mar 21, 2026
Merged

more options for config file generation#1314
phlptp merged 13 commits into
CLIUtils:mainfrom
phlptp:defaults_for_inactive_subcommands

Conversation

@phlptp
Copy link
Copy Markdown
Collaborator

@phlptp phlptp commented Mar 19, 2026

Add a way to control whether only active subcommand defaults are written to a config file.

@phlptp
Copy link
Copy Markdown
Collaborator Author

phlptp commented Mar 19, 2026

Fixes #541

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (e4ee3af) to head (a7bbf03).
⚠️ Report is 188 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##              main     #1314     +/-   ##
===========================================
  Coverage   100.00%   100.00%             
===========================================
  Files           17        19      +2     
  Lines         4546      5139    +593     
  Branches         0      1100   +1100     
===========================================
+ Hits          4546      5139    +593     

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

@phlptp
Copy link
Copy Markdown
Collaborator Author

phlptp commented Mar 20, 2026

@henryiii I either need to be given permissions to modify the configuration on codacy for CLI11 or you need to disable the missing include check or I am going to disable all the codacy check. Right now it is not useful given the number of false positives it is generating.

@henryiii
Copy link
Copy Markdown
Collaborator

I disabled the pattern on one of them, I think/hope that will disable all of them on the next run. AFAICT you should have access already, if you sign in with GitHub; I think it just uses the GitHub permissions?

@phlptp
Copy link
Copy Markdown
Collaborator Author

phlptp commented Mar 20, 2026

I can see everything but I can't change the settings for the projects as best as I can tell. On my other projects I have access to a code_patterns tab and settings tab in Codacy where I can change that but on cli11 I can just see issues. Hopefully that change propagates and it is a non-issue going forward.

@phlptp
Copy link
Copy Markdown
Collaborator Author

phlptp commented Mar 21, 2026

I will merge tomorrow morning unless there are objections or comments.

@phlptp phlptp merged commit 7800246 into CLIUtils:main Mar 21, 2026
74 checks passed
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.

2 participants