D2108: infinitepush: drop the `--to` flag to push and use `-B` instead

2018-03-30 Thread pulkit (Pulkit Goyal)
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

D2108: infinitepush: drop the `--to` flag to push and use `-B` instead

2018-03-27 Thread pulkit (Pulkit Goyal)
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

D2108: infinitepush: drop the `--to` flag to push and use `-B` instead

2018-02-15 Thread pulkit (Pulkit Goyal)
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.

D2108: infinitepush: drop the `--to` flag to push and use `-B` instead

2018-02-12 Thread indygreg (Gregory Szorc)
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

D2108: infinitepush: drop the `--to` flag to push and use `-B` instead

2018-02-12 Thread durham (Durham Goode)
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

D2108: infinitepush: drop the `--to` flag to push and use `-B` instead

2018-02-09 Thread indygreg (Gregory Szorc)
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

D2108: infinitepush: drop the `--to` flag to push and use `-B` instead

2018-02-09 Thread pulkit (Pulkit Goyal)
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

D2108: infinitepush: drop the `--to` flag to push and use `-B` instead

2018-02-09 Thread pulkit (Pulkit Goyal)
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