Page MenuHomePhabricator

Add "does not match regexp" to Herald
ClosedPublic

Authored by epriestley on Feb 11 2016, 11:21 PM.
Tags
None
Referenced Files
F17681333: D15254.id36808.diff
Sun, Jul 13, 6:20 PM
F17678985: D15254.id36809.diff
Sun, Jul 13, 2:09 PM
F17664936: D15254.id.diff
Sat, Jul 12, 7:46 PM
F17647248: D15254.diff
Sat, Jul 12, 1:29 AM
Unknown Object (File)
Jun 26 2025, 1:26 PM
Unknown Object (File)
Jun 1 2025, 1:03 PM
Unknown Object (File)
May 31 2025, 6:17 PM
Unknown Object (File)
May 30 2025, 7:07 PM
Subscribers
None

Details

Summary

Fixes T10330.

  • Anywhere we support "matches regexp", also allow "does not match regexp". Although you can sometimes write a clever negative regexp, these rules are better expressed with "does not match <simple regexp>" anyway, and sometimes no regexp will work.
  • Always allow "does not contain" when we support "contains".
  • Fix some JS issues with certain rules affecting custom fields.
Test Plan
  • Wrote an "Affected files do not match regexp" rule that required every diff to touch "MANUALCHANGELOG.md".
  • Tried to diff without the file; rejected.
  • Tried to diff with the file; accepted.
  • Wrote a bunch of "contains" and "does not contain" rules against text fields and custom fields, then edited tasks to trigger/observe them.
  • Swapped the editor into custom text, user, remarkup, etc fields, no more JS errors.

Screen Shot 2016-02-11 at 3.17.21 PM.png (509×1 px, 68 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Add "does not match regexp" to Herald.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Feb 11 2016, 11:22 PM
This revision was automatically updated to reflect the committed changes.