Page MenuHomePhabricator
Differential D15907

Don't trigger audits for archived packages
ClosedPublic
Actions

Authored by epriestley on May 13 2016, 12:50 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 17, 8:28 PM
Unknown Object (File)
Sun, Dec 8, 8:42 PM
Unknown Object (File)
Sun, Dec 8, 5:02 AM
Unknown Object (File)
Thu, Dec 5, 2:34 PM
Unknown Object (File)
Sat, Nov 30, 6:13 PM
Unknown Object (File)
Wed, Nov 27, 7:40 AM
Unknown Object (File)
Fri, Nov 22, 11:09 PM
Unknown Object (File)
Nov 18 2024, 6:16 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T10939: Support for OWNERS files
Commits
rPe5f2ccc57fb8: Don't trigger audits for archived packages
Summary

Ref T10939. This is just a bug. I thought this was what was described in T10174 but that's actually talking about something completely different.

Also make a <select /> slightly easier to use.

Test Plan
  • Created a package with auditing enabled.
  • Pushed a change.
  • Saw audit trigger.
  • Disabled the package, pushed a change.
    • Before patch: saw audit trigger improperly.
    • After patch: restarted daemons, then saw audit correctly not trigger.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 38309.May 13 2016, 12:50 PM
epriestley retitled this revision from to Don't trigger audits for archived packages.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T10939: Support for OWNERS files.
chad accepted this revision.May 13 2016, 12:52 PM
chad edited edge metadata.
This revision is now accepted and ready to land.May 13 2016, 12:52 PM
Closed by commit rPe5f2ccc57fb8: Don't trigger audits for archived packages (authored by epriestley, committed by epriestley). · Explain WhyMay 13 2016, 1:49 PM
This revision was automatically updated to reflect the committed changes.
epriestley mentioned this in T9491: When packages are archived, packages shouldn't be listed as auditor.May 14 2016, 12:36 AM
epriestley mentioned this in T10181: Create audit requests based on a package commits.May 17 2016, 12:53 PM

Revision Contents
Changeset List

PathSize
src/
applications/
owners/
controller/
2 lines
repository/
worker/
5 lines

Diff 38311

View Options

src/applications/owners/controller/PhabricatorOwnersPathsController.php

Loading...
View Options

src/applications/repository/worker/PhabricatorRepositoryCommitOwnersWorker.php

Loading...
Phabricator · Built by Phacility