This revision was automatically updated to reflect the committed changes.
Closed by commit rHGc5687ce3b411: infinitepush: drop the `--to` flag to push
and use `-B` instead (authored by pulkit, committed by ).
CHANGED PRIOR TO COMMIT
https://phab.mercurial-scm.org/D2108?vs=7334=7422#toc
pulkit updated this revision to Diff 7334.
REPOSITORY
rHG Mercurial
CHANGES SINCE LAST UPDATE
https://phab.mercurial-scm.org/D2108?vs=5369=7334
REVISION DETAIL
https://phab.mercurial-scm.org/D2108
AFFECTED FILES
hgext/infinitepush/__init__.py
tests/test-infinitepush-bundlestore.t
pulkit added a comment.
In https://phab.mercurial-scm.org/D2108#35150, @indygreg wrote:
> I'm find with you sending more parts to this series. Especially if they
delete code: if they delete code then my review of the import will be a review
of the final state of the code post deletions.
indygreg added a comment.
In https://phab.mercurial-scm.org/D2108#36335, @durham wrote:
> > There are things which I am not sure whether to keep or not:
> >
> > - the --bundle-store flag to push command
>
> This is useful for scripts or tools that want to upload a commit to the
durham added a comment.
> There are things which I am not sure whether to keep or not:
>
> - the --bundle-store flag to push command
This is useful for scripts or tools that want to upload a commit to the cloud
without having to give it a name. For instance, you can use it to push a
indygreg accepted this revision.
indygreg added a comment.
This revision is now accepted and ready to land.
I'm find with you sending more parts to this series. Especially if they
delete code: if they delete code then my review of the import will be a review
of the final state of the code
pulkit added a subscriber: durham.
pulkit added a comment.
There is still some work required to make this extension better. I didn't
want to grow my stack too long and hence send upto here. Few things which will
be coming for sure:
- removal of wrapping of `push` command on client side
pulkit created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
The extension added a `--to` flag to specify the bookmark to which revs should
be pushed. This patch deletes that flag and instead uses the `-B` flag. After
this