Page MenuHomePhabricator
Differential D20464

Remove "--force-autoclose" from "bin/repository reparse"
ClosedPublic
Actions

Authored by epriestley on Apr 22 2019, 11:10 PM.
Tags
None
Referenced Files
F15582258: D20464.id48818.diff
Wed, May 7, 3:12 PM
F15579860: D20464.id.diff
Wed, May 7, 2:01 AM
F15553598: D20464.id.diff
Mon, Apr 28, 2:43 AM
F15539102: D20464.id48818.diff
Thu, Apr 24, 11:17 PM
F15534627: D20464.id48818.diff
Thu, Apr 24, 1:05 AM
F15526661: D20464.id.diff
Mon, Apr 21, 10:09 PM
F15522258: D20464.diff
Sun, Apr 20, 7:05 PM
F15504478: D20464.id48818.diff
Mon, Apr 14, 6:07 PM
View All Files
Subscribers
None

Details

Reviewers
amckinley
Maniphest Tasks
T13277: In repositories, realign "Track Only", "Autoclose", and "Publish/Notify" toward "Permanent Refs"
Commits
rP45b9859f0229: Remove "--force-autoclose" from "bin/repository reparse"
Summary

Depends on D20463. Ref T13277. This flag was added some time before 2015 and I don't think I've ever used it. Just get rid of it.

Test Plan

Grepped for force-autoclose, forceAutoclose, AUTOCLOSE_FORCED.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 22 2019, 11:10 PM
Harbormaster completed remote builds in B22690: Diff 48818.Apr 22 2019, 11:11 PM
epriestley requested review of this revision.Apr 22 2019, 11:11 PM
epriestley added inline comments.Apr 22 2019, 11:17 PM
src/applications/repository/worker/commitmessageparser/PhabricatorRepositoryCommitMessageParserWorker.php
175–178

(This was really mooted by the last change; there's a longer explanatory comment in the new worker now.)

epriestley added a child revision: D20465: Remove nearly all remaining references to "Autoclose".Apr 23 2019, 12:12 AM
amckinley accepted this revision.Apr 24 2019, 12:32 AM
This revision is now accepted and ready to land.Apr 24 2019, 12:32 AM
Closed by commit rP45b9859f0229: Remove "--force-autoclose" from "bin/repository reparse" (authored by epriestley). · Explain WhyApr 24 2019, 3:24 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
diffusion/
editor/
6 lines
repository/
management/
8 lines
storage/
1 line
worker/
commitmessageparser/
15 lines

Diff 48838

View Options

src/applications/diffusion/editor/DiffusionCommitEditEngine.php

Loading...
View Options

src/applications/repository/management/PhabricatorRepositoryManagementReparseWorkflow.php

Loading...
View Options

src/applications/repository/storage/PhabricatorRepository.php

Loading...
View Options

src/applications/repository/worker/commitmessageparser/PhabricatorRepositoryCommitMessageParserWorker.php

Loading...
Phabricator · Built by Phacility