Skip to content
Snippets Groups Projects
  1. Aug 22, 2017
  2. Aug 28, 2017
  3. Aug 23, 2017
  4. Aug 25, 2017
  5. Aug 26, 2017
  6. Aug 25, 2017
    • Jun Wu's avatar
      phabsend: show associated Differential Revisions with --confirm · 088598153aa2
      Jun Wu authored
      Often people running `phabsend --confirm` just want to check whether a
      commit will trigger a creation of new Differential Revision, or update an
      existing one. This patch implements that. The `--confirm` message was
      changed to use node instead of revision number to be consistent with what
      `phabsend` outputs.
      
      An example output looks like:
      
        D487 - a80f447973a0 test-extension: enable demandimport explicitly
        D494 - cf440ea6e47e test-casecollision-merge: fix the test
        NEW - 0a6b97147128 phabsend: polish the docstring a bit
        Send the above changes to https://phab.mercurial-scm.org/ (yn)?
      
      Differential Revision: https://phab.mercurial-scm.org/D514
      088598153aa2
    • Jun Wu's avatar
      phabsend: print the actual URL with --confirm · edeb8f28c031
      Jun Wu authored
      Sometimes people have multiple Phabricator endpoints set in multiple repos.
      It seems better for `--confirm` to prompt about the Phabricator endpoint
      patches being sent to.
      
      Differential Revision: https://phab.mercurial-scm.org/D513
      edeb8f28c031
    • Jun Wu's avatar
      phabsend: detect patch change with larger context · 27ff2a87d8c0
      Jun Wu authored
      Previously phabsend has an optimization that will skip uploading a diff if
      the patch (with context line number = 1) remains unchanged. That could be
      confusing:
      
        Aug 24 15:52:28 <martinvonz> phillco: something is wrong with phabricator'your patches/
        Aug 24 15:52:45 <martinvonz> ... with phabricator's view of your patches again
        Aug 24 15:53:38 <martinvonz> if i phabread D388 and then D399, i get a version of filemerge.py with "a, b, c" somewhere on line 344, which is not what phabricator shows for D399
        Aug 24 15:53:51 <martinvonz> junw: maybe that's more for you ^
      
      Fix that by checking context with 32767 lines, which is the same as what
      will be actually sent.
      
      Differential Revision: https://phab.mercurial-scm.org/D512
      27ff2a87d8c0
  7. Aug 24, 2017
    • Jun Wu's avatar
      phabsend: make --amend the default · 07ffff841863
      Jun Wu authored
      The local tag feature was intended to make `phabsend` closer to `email`
      workflow. But its experience is not great in multiple ways:
      
        - after rebase, obsoleted changesets are still visible because of tags
        - without obsstore, the association information will get lost
        - even with obsstore, things could go wrong with graft, export+import
        - no easy way to tell which Differential Revision a commit is associated
      
      Therefore make `--amend` the default. People wanting the old behavior can
      use `--no-amend`.
      
      Differential Revision: https://phab.mercurial-scm.org/D511
      07ffff841863
  8. Aug 25, 2017
  9. Aug 28, 2017
    • Peter Vitt's avatar
      record: make the m key open an editor for the commit message (issue5667) · a1cd6eae2ad3
      Peter Vitt authored
      With the former crecord extension, the user could edit the commit
      message while he was de-/selecting hunks. By pressing 'm', an editor
      showed up to edit the commit message.
      
      With record being part of mercurial, this feature is not available
      anymore. However, the help text still mentions it.
      
      As the infrastructure needed is still present, this feature is quite
      easily ported from the crecord extension to mercurial.
      
      It seems there is no test coverage for record ui, so I tested this patch
      manually on my local machine.
      a1cd6eae2ad3
  10. Aug 22, 2017
Loading