Diffusion Phabricator ce7c2097b216

Update Owners docs a bit
ce7c2097b216
Actions

Description

Update Owners docs a bit

Summary:
Fixes T9218. Fixes T8320. Fixes T8661. This isn't exhaustive but documents the stuff that cropped up in this iteration as needing documentation. In particular:

Be explicit about multiple ownership.
Explain value of having one place to update your giant regexp of a trillion paths.

Test Plan: Read documentation.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T8320, T8661, T9218

Differential Revision: https://secure.phabricator.com/D14023

Details

Provenance

epriestley	Authored on
epriestley	Pushed on Aug 31 2015, 11:01 PM

Reviewer

chad

Differential Revision

D14023: Update Owners docs a bit

Parents

rPfe66d52a22a5: Don't copy `null` attributes passed to JX.$N()

Branches

Unknown

Tags

Unknown

Tasks

T8320: Bring Owners out of the dark ages
T8661: Unclear distinction between Herald rule and Owners package
T9218: Should Owners allow multiple packages to own the same file?

Event Timeline

epriestley committed rPce7c2097b216: Update Owners docs a bit (authored by epriestley).Aug 31 2015, 11:01 PM

epriestley added a task: T9218: Should Owners allow multiple packages to own the same file?.

epriestley added a task: T8661: Unclear distinction between Herald rule and Owners package.

epriestley added a task: T8320: Bring Owners out of the dark ages.

• chenxiruanhai added a subscriber: • chenxiruanhai.Sep 1 2015, 9:03 AM

• chenxiruanhai added inline comments.

/src/docs/user/userguide/owners.diviner
35	test line comment.

Changes (2)

Path

Size

src/

applications/

owners/

application/

PhabricatorOwnersApplication.php

docs/

user/

userguide/

owners.diviner

rPce7c2097b216

View Options

src/applications/owners/application/PhabricatorOwnersApplication.php