Page MenuHomePhabricator

Apply "enormous changes" rules to pre-commit content rules too
ClosedPublic

Authored by epriestley on Jan 3 2014, 9:33 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 29, 7:35 PM
Unknown Object (File)
Mon, Apr 29, 7:35 PM
Unknown Object (File)
Mon, Apr 29, 7:35 PM
Unknown Object (File)
Mon, Apr 29, 3:18 PM
Unknown Object (File)
Thu, Apr 25, 8:05 PM
Unknown Object (File)
Wed, Apr 24, 10:23 PM
Unknown Object (File)
Sat, Apr 20, 6:16 PM
Unknown Object (File)
Fri, Apr 12, 12:55 AM
Subscribers

Details

Summary

Fixes T4276. This adds "Change is enormous" to pre-commit content rules so we can, e.g., just reject these and not worry about them elsewhere.

Also, use the same numeric limits across the mechanisms so there's a consistent definition of an "enormous" changeset.

Test Plan
  • Set enormous limit to 15 bytes, pushed some changes, got blocked by a rule.
  • Set it back, pushed OK.

Diff Detail

Branch
enormpre
Lint
Lint Passed
Unit
Tests Passed