Ref T7674. This linter doesn't make sense without commit hooks.
Details
Details
- Reviewers
epriestley - Group Reviewers
Blessed Reviewers - Maniphest Tasks
- T7674: Remove support for arc commit hooks
- Commits
- rARC0846c6aff5c4: Remove commit linter
N/A
Diff Detail
Diff Detail
- Repository
- rARC Arcanist
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Comment Actions
This is still sort of useful, in that you can write something like this, right?
// @nocommit: Just for testing, revert before sending for review!! // $value = real_value(); $value = 42;
I think it's still fine to get rid of since this is super marginal and you can write a similar rule using script+regex or something once it modernizes.
Comment Actions
You could do that, but it doesn't prevent the code from actually being pushed. This makes it significantly less valuable.