Paths

Table of Contentst

Differential D19426

Prepare to support an "Ignore generated files" flag in Owners
ClosedPublic
Actions

Authored by epriestley on May 4 2018, 8:30 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Jan 28, 3:50 AM

	Unknown Object (File)
	Sat, Jan 25, 1:24 AM

	Unknown Object (File)
	Sat, Jan 25, 1:24 AM

	Unknown Object (File)
	Sat, Jan 25, 1:24 AM

	Unknown Object (File)
	Thu, Jan 23, 5:52 AM

	Unknown Object (File)
	Thu, Jan 23, 3:27 AM

	Unknown Object (File)
	Tue, Jan 21, 9:47 AM

	Unknown Object (File)
	Fri, Jan 17, 9:07 PM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T13130: Plans: 2018 Week 17/18 Bonus Content

Commits

rP5e2af4b9b5cd: Prepare to support an "Ignore generated files" flag in Owners

Summary

Depends on D19425. Ref T13130. See PHI251. Now that changesets have a durable "generated" attribute, we can let owners packages check it when we're computing which packages are affected by a revision.

There's no way to actualy configure a package to have this behavior yet.

Test Plan

Created a revision affecting a generated file and a non-generated file.
- When I faked mustMatchUngeneratedPaths() to return true;, saw the non-generated file get no packages owning it.
- Normally: lots of packages owning it).
Created a revision affecting only generated files.
- When I faked things, saw no Owners actions trigger.
- Normally: some packages added reviewers or subscribers.

Diff Detail

Repository

Branch

gen2

Lint

Lint Passed

Severity	Location	Code	Message
Advice	src/applications/owners/storage/PhabricatorOwnersPackage.php:151	XHP16	TODO Comment

Unit

Tests Passed

Build Status

Buildable 20277
Build 27525: Run Core Tests
Build 27524: arc lint + arc unit

Event Timeline

epriestley created this revision.May 4 2018, 8:30 PM

Harbormaster completed remote builds in B20277: Diff 46467.May 4 2018, 8:31 PM

epriestley requested review of this revision.May 4 2018, 8:31 PM

epriestley added a child revision: D19427: Remove explicit "mailKey" from Owners packages.May 4 2018, 8:44 PM

amckinley accepted this revision.May 5 2018, 2:22 AM

This revision is now accepted and ready to land.May 5 2018, 2:22 AM

Closed by commit rP5e2af4b9b5cd: Prepare to support an "Ignore generated files" flag in Owners (authored by epriestley). · Explain WhyMay 5 2018, 3:46 PM

This revision was automatically updated to reflect the committed changes.

epriestley mentioned this in D21614: Lift peculiar side effect of path indexing out of indexer.Mar 15 2021, 5:50 PM

epriestley mentioned this in rPc317d16bdd56: Lift peculiar side effect of path indexing out of indexer.Mar 15 2021, 11:16 PM

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

controller/

DifferentialController.php

10 lines

editor/

DifferentialTransactionEditor.php

26 lines

herald/

HeraldDifferentialRevisionAdapter.php

5 lines

storage/

DifferentialDiff.php

18 lines

owners/

storage/

PhabricatorOwnersPackage.php

82 lines

Diff 46467

src/applications/differential/controller/DifferentialController.php

Loading...

src/applications/differential/editor/DifferentialTransactionEditor.php

Loading...

src/applications/differential/herald/HeraldDifferentialRevisionAdapter.php

Loading...

src/applications/differential/storage/DifferentialDiff.php

Loading...

src/applications/owners/storage/PhabricatorOwnersPackage.php

Loading...