Page MenuHomePhabricator
Differential D20427

Deprecate the "Commit is on autoclose/permanent branch" Herald "Commit" field
ClosedPublic
Actions

Authored by epriestley on Apr 15 2019, 4:48 PM.
Tags
None
Referenced Files
F15468458: D20427.diff
Fri, Apr 4, 12:59 AM
F15460445: D20427.diff
Mon, Mar 31, 11:06 PM
F15459872: D20427.id.diff
Mon, Mar 31, 5:31 PM
F15429430: D20427.id.diff
Mon, Mar 24, 2:13 AM
F15416345: D20427.id48729.diff
Thu, Mar 20, 10:35 AM
F15381372: D20427.id48783.diff
Fri, Mar 14, 7:18 AM
F15377443: D20427.id48729.diff
Thu, Mar 13, 8:53 AM
F15283283: D20427.diff
Mar 4 2025, 6:35 AM
View All Files
Subscribers
None

Details

Reviewers
amckinley
Maniphest Tasks
T13277: In repositories, realign "Track Only", "Autoclose", and "Publish/Notify" toward "Permanent Refs"
Commits
rP9107c2e26208: Deprecate the "Commit is on autoclose/permanent branch" Herald "Commit" field
Summary

Depends on D20426. Ref T13277. The new behavior is to fire Herald only once a commit becomes reachable from a permanent ref (previously, an "Autoclose" branch).

That means that every "Commit" Herald rule implicitly has this field as a condition, and it no longer does anything.

Test Plan

Wrote a Herald rule, saw this as an option in the "Deprecated" section.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 15 2019, 4:48 PM
Harbormaster completed remote builds in B22599: Diff 48729.Apr 15 2019, 4:49 PM
epriestley requested review of this revision.Apr 15 2019, 4:49 PM
epriestley added a child revision: D20428: When a commit appears as an ancestor of a permanent ref for the first time, run all import steps.Apr 15 2019, 5:04 PM
epriestley mentioned this in T13041: In Herald Commit rules, "Reviewer" and "On Autoclose Branch" have been deprecated.Apr 15 2019, 5:05 PM
amckinley accepted this revision.Apr 17 2019, 8:08 PM

I guess it's not worth cleaning this up and removing the class + DB rows since it's just a no-op now and shouldn't bother anyone.

This revision is now accepted and ready to land.Apr 17 2019, 8:08 PM
epriestley added a comment.Apr 17 2019, 8:14 PM

I think it's probably worth cleaning up at some point -- there's another similar rule in T13041, too, that could do with cleanup at the same time. But it's low/no-cost to just leave it around for a while and let users mostly migrate away from it on their own, hopefully.

Closed by commit rP9107c2e26208: Deprecate the "Commit is on autoclose/permanent branch" Herald "Commit" field (authored by epriestley). · Explain WhyApr 18 2019, 12:32 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
diffusion/
herald/
10 lines

Diff 48783

View Options

src/applications/diffusion/herald/DiffusionCommitAutocloseHeraldField.php

Loading...
Phabricator · Built by Phacility