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
F18782533: D7887.id.diff
Mon, Oct 13, 1:52 AM
F18771115: D7887.id17846.diff
Wed, Oct 8, 5:42 PM
F18740927: D7887.id17866.diff
Thu, Oct 2, 7:02 AM
F18694004: D7887.diff
Sat, Sep 27, 2:16 AM
F18675615: D7887.diff
Thu, Sep 25, 5:55 PM
F18671117: D7887.diff
Thu, Sep 25, 2:44 AM
F18633663: D7887.diff
Sep 16 2025, 8:50 PM
F18627792: D7887.id.diff
Sep 16 2025, 4:47 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

Lint
Lint Skipped
Unit
Tests Skipped