-
Notifications
You must be signed in to change notification settings - Fork 14
New commands: add
and add-from-fs
#465
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
base: master
Are you sure you want to change the base?
Conversation
Reviewer's GuideThis pull request introduces two new CLI subcommands, File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
add
and add-from-fs
add
and add-from-fs
a36fbef
to
69a222e
Compare
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #465 +/- ##
==========================================
- Coverage 78.93% 76.70% -2.24%
==========================================
Files 8 11 +3
Lines 413 807 +394
Branches 85 184 +99
==========================================
+ Hits 326 619 +293
- Misses 52 119 +67
- Partials 35 69 +34 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
d37cb7e
to
628830e
Compare
e85482d
to
e26036d
Compare
740c98c
to
1045626
Compare
b02d442
to
27d3297
Compare
f310106
to
9c5c92e
Compare
5149ebd
to
095d919
Compare
0499067
to
f67766f
Compare
1be9760
to
81eb435
Compare
…g fixes why: - expose configuration management (add, add-from-fs, fmt) through the main CLI - keep user-facing output visible after the logging refactor - ensure default config readers use UTF-8 consistently what: - wire add/add-from-fs/fmt subcommands into `vcspull` CLI entry and parser - implement `cli/add.py`, `cli/add_from_fs.py`, `cli/fmt.py` plus extensive tests - centralize config saves via `save_config_yaml` and open configs as UTF-8 - refactor `setup_logger` to replace NullHandler with stdout stream and add regression test for CLI output - expand logging test suite to cover formatter and propagation behavior This keeps the subject succinct while the bullets break down the “why/what” of the large diff. Adjust component prefixes as needed if you prefer more granularity (e.g., separate commits per concern), but this structure documents all the key changes in one place.
Changes
New: Added
vcspull fmt
fmt
subcommand to normalize configuration files (sort directories/repos, convert compact entries to verbose{repo: ...}
form, and standardize keys).uv run vcspull fmt --config ~/.vcspull.yaml --write
New: Added
vcspull add
uv run vcspull add myrepo [email protected]:user/myrepo.git --dir ~/code
New: Added
vcspull add-from-fs
--yes
is specified.uv run vcspull add-from-fs ~/study/typescript --config ~/.vcspull.yaml --yes
Logging Improvements
test_add_from_fs_stream_output
to guarantee user-facing output remains visible.Config & CLI Enhancements
ConfigReader
now reads files using UTF-8 explicitly.save_config_yaml
centralizes YAML writes with UTF-8 encoding.Fixes: #25, #333
Testing
uv run ruff check src/vcspull/log.py tests/cli/test_add_from_fs.py
uv run mypy
uv run py.test
Additional Notes