Page MenuHomePhabricator

Remove nearly all remaining references to "Autoclose"
ClosedPublic

Authored by epriestley on Apr 23 2019, 12:12 AM.
Tags
None
Referenced Files
F19818387: D20465.diff
Thu, Mar 5, 8:19 PM
F19813902: D20465.diff
Wed, Mar 4, 11:32 AM
F19793996: D20465.id.diff
Thu, Feb 26, 6:15 PM
F19790283: D20465.diff
Wed, Feb 25, 8:13 PM
F19507933: D20465.id48839.diff
Jan 9 2026, 11:58 PM
F19507932: D20465.id48819.diff
Jan 9 2026, 11:58 PM
F19507931: D20465.id.diff
Jan 9 2026, 11:58 PM
F19210717: D20465.id48839.diff
Dec 17 2025, 1:30 AM
Subscribers
None

Details

Summary

Depends on D20464. Ref T13277. Broadly:

  • Move all the "should publish X" and "why aren't we publishing X" stuff to a separate class (PhabricatorRepositoryPublisher).
  • Rename things to be more consistent with modern terminology ("Publish", "Permanent Refs").
Test Plan

This could use some trial-by-fire on secure, but:

  • Grepped for all symbols.
  • Viewed various commits.
  • Reparsed commits.
  • Here's a commit with an explanation:

Screen Shot 2019-04-22 at 5.08.48 PM.png (106×501 px, 14 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable