Allow `jj git push` to push multiple branches/changes at once #912

ilyagr · 2022-12-18T00:39:22Z

Also creates a short arg -b for --branch.

I haven't tried to make the code very pretty, but I got tests to pass.

Checklist

If applicable:

I have updated CHANGELOG.md
Only command-line help is planned for documentation
I have added tests to cover my changes

ilyagr · 2022-12-18T00:46:21Z

Ideally, i'd want to do something like jj git push -r main:@ to push all the branches in between @ and main, but that's a larger change that might need some thinking about the UI.

src/commands.rs

CHANGELOG.md

tests/test_git_push.rs

src/commands.rs

tests/test_git_push.rs

src/commands.rs

tests/test_git_push.rs

Also creates short arg `-b` for `--branch`.

martinvonz

Thanks

ilyagr force-pushed the ig/push_many branch from 9ba1f40 to a90e6db Compare December 18, 2022 00:40

ilyagr changed the title ~~Ig/push many~~ Allow jj git push to push multiple branches/changes at once Dec 18, 2022

ilyagr force-pushed the ig/push_many branch from a90e6db to 3919af9 Compare December 18, 2022 03:15

yuja reviewed Dec 18, 2022

View reviewed changes

src/commands.rs Outdated Show resolved Hide resolved

src/commands.rs Outdated Show resolved Hide resolved

src/commands.rs Show resolved Hide resolved

martinvonz reviewed Dec 18, 2022

View reviewed changes

src/commands.rs Outdated Show resolved Hide resolved

src/commands.rs Outdated Show resolved Hide resolved

CHANGELOG.md Show resolved Hide resolved

ilyagr force-pushed the ig/push_many branch from 3919af9 to db12e70 Compare December 18, 2022 23:50

ilyagr enabled auto-merge (rebase) December 18, 2022 23:55

martinvonz reviewed Dec 19, 2022

View reviewed changes

ilyagr force-pushed the ig/push_many branch 5 times, most recently from 15f35d1 to c542f8d Compare December 19, 2022 02:30

martinvonz reviewed Dec 19, 2022

View reviewed changes

tests/test_git_push.rs Outdated Show resolved Hide resolved

ilyagr force-pushed the ig/push_many branch from c542f8d to 92a2cce Compare December 20, 2022 07:25

Add a test for git push --change

c9a57dd

ilyagr force-pushed the ig/push_many branch from 92a2cce to b75326b Compare December 20, 2022 07:48

ilyagr commented Dec 20, 2022

View reviewed changes

tests/test_git_push.rs Outdated Show resolved Hide resolved

martinvonz reviewed Dec 20, 2022

View reviewed changes

src/commands.rs Outdated Show resolved Hide resolved

tests/test_git_push.rs Outdated Show resolved Hide resolved

ilyagr added 3 commits December 20, 2022 17:07

Allow jj git push to push multiple branches/changes at once

efec235

Also creates short arg `-b` for `--branch`.

jj git push: Make sure each branch is processed at most once

fadd8ef

Rearrange branches of if statement in jj git push

021a6e4

ilyagr force-pushed the ig/push_many branch from b75326b to 021a6e4 Compare December 21, 2022 01:17

martinvonz approved these changes Dec 21, 2022

View reviewed changes

ilyagr merged commit 0285a20 into main Dec 21, 2022

ilyagr deleted the ig/push_many branch December 21, 2022 08:52

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Allow `jj git push` to push multiple branches/changes at once #912

Allow `jj git push` to push multiple branches/changes at once #912

ilyagr commented Dec 18, 2022 •

edited

Loading

ilyagr commented Dec 18, 2022

martinvonz left a comment

Allow jj git push to push multiple branches/changes at once #912

Allow jj git push to push multiple branches/changes at once #912

Conversation

ilyagr commented Dec 18, 2022 • edited Loading

Checklist

ilyagr commented Dec 18, 2022

martinvonz left a comment

Choose a reason for hiding this comment

Allow `jj git push` to push multiple branches/changes at once #912

Allow `jj git push` to push multiple branches/changes at once #912

ilyagr commented Dec 18, 2022 •

edited

Loading