See PHI370. Support the "Affected packages" and "Affected package owners" Herald fields in pre-commit hooks.
I believe there's no technical reason these fields aren't supported and this was just overlooked.
Details
Details
- Reviewers
- None
- Commits
- rPf713e1dfc160: Add Owners Package support for "Commit Hook: Content" Herald rules
Wrote a rule which makes use of the new fields, pushed commits through it. Checked transcripts and saw sensible-looking values.
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Branch
- precommit1
- Lint
Lint Passed - Unit
Tests Passed - Build Status
Buildable 19586 Build 26512: Run Core Tests Build 26511: arc lint + arc unit