Page MenuHomePhabricator

Bring Owners out of the dark ages
Closed, ResolvedPublic


Owners is very badly out of date, and T7604 at least partially motivates spending some time modernizing it.

Event Timeline

epriestley updated the task description. (Show Details)
epriestley raised the priority of this task from to Normal.
epriestley claimed this task.
epriestley added a project: Owners.
epriestley added a subtask: Restricted Maniphest Task.
epriestley moved this task from Backlog to Dark Ages on the Owners board.
chad awarded a token.May 26 2015, 7:11 PM

What does this have to do with T7604?

Installs which relied on "Arcanist Project" Herald rules don't necessarily have a perfect way forward; "Owners"-based rules are one of the alternatives I suggest in the upgrading guide in the description of that task.

So, anyone have any clue what we use "primary owner" for?

As per IRC:

[20:56:34] <_richardJ> I have a commit for which an Owners package of which I'm an owner requires an audit because of "owners not involved". However, I reviewed the actual revision.
eadler added a subscriber: eadler.Jun 4 2015, 8:25 PM
nickz added a subscriber: nickz.Jun 10 2015, 10:21 PM
epriestley moved this task from Backlog to The Queue on the Prioritized board.
turadg added a subscriber: turadg.Aug 25 2015, 11:36 PM

I marked D14023 as resolving this since I think I took care of most of the really silly/old stuff and this now works in a fairly reasonable way, at least for the most part.

I sketched out a very rough v2 on the workboard covering T2443 / T8425. The former should wait for API stuff and the latter is just a messy migration thing. Depending on when Audit gets a round of updates that stuff might make it in, too. I'm mostly hoping to shelve this for a bit, though, if nothing too sketchy pops up.