Page MenuHomePhabricator

Remove commit linter
ClosedPublic

Authored by joshuaspence on May 4 2015, 2:51 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 22, 3:05 AM
Unknown Object (File)
Wed, Dec 18, 1:05 AM
Unknown Object (File)
Mon, Dec 16, 5:18 AM
Unknown Object (File)
Thu, Dec 5, 9:27 AM
Unknown Object (File)
Thu, Nov 28, 4:40 PM
Unknown Object (File)
Thu, Nov 28, 4:40 PM
Unknown Object (File)
Thu, Nov 28, 4:40 PM
Unknown Object (File)
Thu, Nov 28, 4:26 PM
Subscribers

Details

Summary

Ref T7674. This linter doesn't make sense without commit hooks.

Test Plan

N/A

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

joshuaspence retitled this revision from to Remove commit linter.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.

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.

This revision is now accepted and ready to land.May 4 2015, 2:57 PM

You could do that, but it doesn't prevent the code from actually being pushed. This makes it significantly less valuable.

This revision was automatically updated to reflect the committed changes.