Page MenuHomePhabricator
Differential D20426

Expose repository ref rules via "diffusion.repository.search"
ClosedPublic
Actions

Authored by epriestley on Apr 15 2019, 1:57 PM.
Tags
None
Referenced Files
F13236776: D20426.diff
Tue, May 21, 10:55 AM
F13236491: D20426.diff
Tue, May 21, 9:54 AM
F13235105: D20426.diff
Tue, May 21, 4:15 AM
F13228586: D20426.id48781.diff
Mon, May 20, 10:36 AM
F13212568: D20426.diff
Fri, May 17, 6:53 AM
F13196616: D20426.diff
Sun, May 12, 11:25 PM
F13190028: D20426.diff
Sat, May 11, 7:10 AM
F13178660: D20426.diff
Wed, May 8, 8:34 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
rPaed755e1d88b: Expose repository ref rules via "diffusion.repository.search"
Summary

Depends on D20425. Ref T13277. See PHI1067. There's currently no way to retrieve branch/ref rules over the API, which makes some management operations against a large number of repositories difficult.

Expose these rules to the API.

Test Plan

Called diffusion.repository.search, got rules in the result set.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 15 2019, 1:57 PM
Harbormaster completed remote builds in B22598: Diff 48728.Apr 15 2019, 1:59 PM
epriestley requested review of this revision.Apr 15 2019, 1:59 PM
epriestley added a child revision: D20427: Deprecate the "Commit is on autoclose/permanent branch" Herald "Commit" field.Apr 15 2019, 4:48 PM
amckinley accepted this revision.Apr 17 2019, 8:06 PM
amckinley added inline comments.
src/applications/repository/storage/PhabricatorRepository.php
2842

Should be "Fetch, track, and permanent ref rules", right?

This revision is now accepted and ready to land.Apr 17 2019, 8:06 PM
epriestley added inline comments.Apr 18 2019, 12:27 PM
src/applications/repository/storage/PhabricatorRepository.php
2842

I'm being sneaky about "track" since I'm trying to get rid of it.

Closed by commit rPaed755e1d88b: Expose repository ref rules via "diffusion.repository.search" (authored by epriestley). · Explain WhyApr 18 2019, 12:27 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
repository/
storage/
34 lines

Diff 48781

View Options

src/applications/repository/storage/PhabricatorRepository.php

Loading...
Phabricator · Built by Phacility