Home GnuPG

Add "does not match regexp" to Herald
8934dee5437bUnpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Add "does not match regexp" to Herald

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.

{F1105172}

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T10330

Differential Revision: https://secure.phabricator.com/D15254

Details

Provenance
epriestley <git@epriestley.com>Authored on Feb 12 2016, 12:17 AM
Parents
rPHAB86611708191c: Fix a couple typos in "why so many databases?" document
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHAB8934dee5437b: Add "does not match regexp" to Herald (authored by epriestley <git@epriestley.com>).Feb 12 2016, 12:29 AM