deduplicate CLI subcommands, parser setup, and repo commands - #1737
deduplicate CLI subcommands, parser setup, and repo commands#1737rfbgo wants to merge 2 commits into
Conversation
There was a problem hiding this comment.
Code Review
This pull request refactors subcommand parser setup across multiple command modules (including data, deployment, and workspace) by extracting common setup logic into a new helper function, setup_subcommands_from_prefix, in ramble/cmd/common/__init__.py. It also extracts repository addition and removal configuration logic in repo.py into helper functions, and adds unit tests for the new common functions. Feedback on these changes suggests removing the redundant explicit import of the builtins module and referencing standard built-in types like list and tuple directly rather than prefixing them with builtins..
Ramble Performance Test MetricsResults produced with commit: b7de116
|
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #1737 +/- ##
========================================
Coverage 94.02% 94.03%
========================================
Files 367 368 +1
Lines 36933 36910 -23
========================================
- Hits 34728 34710 -18
+ Misses 2205 2200 -5 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
No description provided.