Page MenuHomePhabricator

phabricator: include bookmark in diff metadata
AbandonedPublic

Authored by Kwan on Oct 10 2019, 5:53 PM.

Details

Reviewers
None
Group Reviewers
hg-reviewers
Summary

While arcanist does this I could see us leaving it out for a couple reasons:

  1. arcanist only operates on the current stack, so can use the actual active bookmark, whereas phabsend works with any commits, so if there are multiple bookmarks it'll just pick the first one arbitrarily
  2. some users might not like the disclosure

Diff Detail

Repository
rHG Mercurial
Branch
creatediff (bookmark) on default (branch)
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

Kwan created this revision.Oct 10 2019, 5:53 PM
Kwan updated this revision to Diff 17086.Oct 11 2019, 1:55 PM

Fix some test-check-code issues, and one test-check-pyflakes unused local.

Kwan updated this revision to Diff 17106.Oct 12 2019, 9:46 AM

I'd missed out the attrs key conversion needed on py3.

Yeah, the bookmark-name disclosure seems like something that should be optional? Maybe put this behind a flag?

(or a config knob)

Kwan abandoned this revision.Wed, Oct 23, 5:20 PM

I'm not that bothered about this so going to abandon it for now, can always revisit it later.