gh19 support checkout not in cli #20
Conversation
BenjiMilan
force-pushed
the
feature/gh19_support_checkout_not_in_cli
branch
from
1ce1e15 to
5d7d8cb
Compare
There was a problem hiding this comment.
Pull request overview
This PR adds support for checking out support branches via the CLI, which was previously missing. The change addresses issue #19 by implementing a checkout command under the support group.
- Added
checkoutcommand to thesupportCLI group to enable switching to a specific support branch - Added missing docstring to the
support startcommand - Updated develop checkout docstring for consistency
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @click.option("-f", "--force", is_flag=True, help="Force checkout.") | ||
| @click.option("-v", "--verify", is_flag=True, help="Run hooks without prompting.") | ||
| def checkout(name, force, verify): | ||
| """Checkout a support branch.""" |
There was a problem hiding this comment.
The docstring ends with a period, but most other checkout commands in this file don't (feature, release, hotfix checkout docstrings have no period). Only master checkout has a period. For consistency, consider removing the period to match the majority pattern.
| """Checkout a support branch.""" | |
| """Checkout a support branch""" |
| @click.option("-v", "--verify", is_flag=True, help="Run hooks without prompting.") | ||
| def checkout(force, verify): | ||
| """Checkout the develop branch""" | ||
| """Checkout a develop branch.""" |
There was a problem hiding this comment.
Changing "the" to "a" makes this less accurate. There is only one develop branch in a git-flow model, unlike feature/release/hotfix/support branches where multiple can exist. The original "Checkout the develop branch" was more accurate. Consider reverting this change.
| """Checkout a develop branch.""" | |
| """Checkout the develop branch.""" |
Support checkout not in cli. Closes #19