Page MenuHomePhabricator

Disable mentions in nonpublishing repositories
ClosedPublic

Authored by epriestley on Mar 4 2015, 2:16 PM.
Tags
None
Referenced Files
F14067420: D11966.id.diff
Tue, Nov 19, 2:57 PM
F14066321: D11966.id.diff
Tue, Nov 19, 9:17 AM
F14010373: D11966.diff
Thu, Oct 31, 7:19 AM
F14008922: D11966.id28815.diff
Wed, Oct 30, 8:01 AM
F13967610: D11966.id28807.diff
Oct 16 2024, 2:45 PM
F13965763: D11966.diff
Oct 16 2024, 4:10 AM
Unknown Object (File)
Oct 8 2024, 5:14 AM
Unknown Object (File)
Sep 21 2024, 7:48 AM
Subscribers

Details

Summary

Ref T6516. Although this behavior is somewhat-arguable as desirable, I think it's less surprising and more consistent to disable mentions when a repository is publishing.

In particular, if you import a repository developed on another Phabricator install, this stops all the T123 in commit messages from creating mentions on your unrelated T123 tasks.

We already disable autoclose, so Closes T123 and Ref T123 already have no effect, but a bare T123 would generate a mention. Likewise, @epriestley would generate a mention.

If you import such a repository and then update it periodically, updates will activate autoclose and publishing (if you didn't disable them), but presumably this will hit a couple of tasks and you'll go change the settings if you forgot.

At some point, we may have some kind of use case for separating the "publish" setting into a "publish" setting and a "this is a local repository" setting. For example, if you work at Widget Corp, want to import Phabricator locally, and want to write Herald rules against it, you can't currently configure the repository to let you do all of this. But we haven't actually seen a use case for this yet.

Test Plan
  • Pushed some commits with bare T11, saw mentions.
  • Disabled publishing for the repository, pushed some commits with
  • Imported a bunch of commits without seeing pipeline failures.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Disable mentions in nonpublishing repositories.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 4 2015, 6:28 PM
This revision was automatically updated to reflect the committed changes.