Homec4science

Allow excluding paths from package

Authored by vrana <jakubv@fb.com> on Dec 7 2012, 02:23.

Description

Allow excluding paths from package

Summary: Resolves T2149.

Test Plan:

$ bin/storage upgrade
  1. /owners/ - saw +
  2. /owners/package/1/ - saw +
  3. /owners/edit/1/ - added exclude paths, saw correct e-mail
  4. /rPabc123 - included paths are still highlighted and excluded not
  5. /owners/view/search/?path=/included/ - found
  6. /owners/view/search/?path=/excluded/ - not found
  7. owners.query - path: /included/
  8. owners.query - path: /excluded/
  9. new unit test

    PhabricatorOwnersPackage::loadAffectedPackages( $repository, array('/excluded/b.php'));

    PhabricatorOwnersPackage::loadAffectedPackages( $repository, array('/included/a.php'));

Reviewers: epriestley

Reviewed By: epriestley

CC: aran, Korvin

Maniphest Tasks: T2149

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

Details

Committed
vrana <jakubv@fb.com>Dec 8 2012, 01:33
Pushed
aubortJan 31 2017, 17:16
Parents
rPHbf9bc885b764: Enable notifications by default
Branches
Unknown
Tags
Unknown

Event Timeline

vrana <jakubv@fb.com> committed rPH4f615ad2a936: Allow excluding paths from package (authored by vrana <jakubv@fb.com>).Dec 8 2012, 01:33