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
F19304032: D20464.diff
Wed, Dec 24, 6:49 AM
F19062903: D20464.id.diff
Nov 29 2025, 4:57 PM
F19062627: D20464.diff
Nov 29 2025, 4:18 PM
F19057518: D20464.id.diff
Nov 28 2025, 10:45 PM
F19050360: D20464.diff
Nov 27 2025, 7:49 PM
F18853131: D20464.id48838.diff
Oct 31 2025, 2:26 PM
F18832772: D20464.id.diff
Oct 25 2025, 10:57 PM
F18829157: D20464.diff
Oct 24 2025, 8:14 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