-
Notifications
You must be signed in to change notification settings - Fork 3
feat: Colored CLI help, sphinx_argparse_neo extension
#337
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #337 +/- ##
===========================================
+ Coverage 75.76% 87.16% +11.39%
===========================================
Files 5 30 +25
Lines 260 2821 +2561
===========================================
+ Hits 197 2459 +2262
- Misses 63 362 +299 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Code reviewNo issues found. Checked for bugs and CLAUDE.md compliance. 🤖 Generated with Claude Code |
16ce786 to
5b20ba8
Compare
Code reviewNo issues found. Checked for bugs and CLAUDE.md compliance. 🤖 Generated with Claude Code |
why: CHANGES file was missing documentation for the major features in PR #337. what: - Add colored CLI help output under What's new - Add py.typed marker under Packaging - Add sphinx_argparse_neo replacement under Development - Add dev dependencies (types-docutils, types-Pygments)
Code reviewNo issues found. Checked for bugs and CLAUDE.md compliance. 🤖 Generated with Claude Code - If this code review was useful, please react with 👍. Otherwise, react with 👎. |
c7a2025 to
9b56918
Compare
Code reviewNo issues found. Checked for bugs and CLAUDE.md compliance. 🤖 Generated with Claude Code - If this code review was useful, please react with 👍. Otherwise, react with 👎. |
987c8ad to
4f7062a
Compare
Code reviewNo issues found. Checked for bugs and CLAUDE.md compliance. 🤖 Generated with Claude Code - If this code review was useful, please react with 👍. Otherwise, react with 👎. |
4f7062a to
b4c45b3
Compare
b4c45b3 to
ade42a8
Compare
Code reviewNo issues found. Checked for bugs and CLAUDE.md compliance. 🤖 Generated with Claude Code - If this code review was useful, please react with 👍. Otherwise, react with 👎. |
why: cihai_cli is the project's own package with py.typed marker what: - Remove cihai_cli and cihai_cli.* from mypy override - Keep only external dependency cihai.* in ignore_missing_imports
why: Copy-paste remnant referenced vcspull instead of cihai-cli what: - Update docstring to reference cihai-cli documentation
Add consistent styling to pre.argparse-usage to match code blocks: - background: #272822 (Monokai, same as .highlight) - padding, line-height, border-radius - thin scrollbars using Furo CSS variable
sphinx_argparse_neo extension
Summary
sphinx-argparsedependency with customsphinx_argparse_neoextensionspy.typedmarker for type checker supportChanges
CLI Enhancements
_colors.pymodule for ANSI color support with auto-detection_formatter.pywithColoredHelpFormatterfor rich help output--helpforinfoandreversecommandsDocumentation Extensions
docs/_ext/sphinx_argparse_neo/moduledocs/_ext/argparse_exemplar.pyextensionsphinx-argparsefrom dependenciesType Safety
py.typedmarker tocihai_clipackagedocs/_ext/types-docutilsandtypes-PygmentsstubsTests
Test plan
uv run ruff check .passesuv run ruff format .passesuv run mypypasses (21 source files)uv run pytest --reruns 0passes (365 tests)sphinx-build -b html docs docs/_build/htmlbuilds successfully