-
Notifications
You must be signed in to change notification settings - Fork 353
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
cli: move: Add -f/-t for --from/--to to jj move
#3062
Conversation
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
9760339
to
8dec5d5
Compare
Be advised that |
I almost wonder if we want to call them |
This seems also related: #2882 |
I can rename to |
I think this PR is good as is. Whether we rename the flags seems like a separate discussion that shouldn't block this.
Yes. |
I generally avoid |
8dec5d5
to
ebe646e
Compare
jj move
jj move
ebe646e
to
7db0c5d
Compare
Fixed the conflict in the changelog |
7db0c5d
to
15dbfb8
Compare
I find myself often using
jj move
when working and having shorthands for--from/--to
would be very usefulChecklist
If applicable:
CHANGELOG.md