Page MenuHomePhabricator

If the Script-and-Regex linter captures no "line" text, treat the message as affecting the entire file
ClosedPublic

Authored by epriestley on Jan 12 2016, 1:24 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 5, 5:25 PM
Unknown Object (File)
Tue, Mar 5, 5:25 PM
Unknown Object (File)
Tue, Mar 5, 5:25 PM
Unknown Object (File)
Tue, Mar 5, 5:24 PM
Unknown Object (File)
Tue, Mar 5, 5:24 PM
Unknown Object (File)
Feb 11 2024, 11:38 AM
Unknown Object (File)
Dec 22 2023, 2:40 AM
Unknown Object (File)
Dec 16 2023, 4:18 AM
Subscribers
None

Details

Summary

Fixes T10124.

Test Plan

Added this "linter" to .arclint:

"thing": {
  "type": "script-and-regex",
  "script-and-regex.script": "echo every file is very bad #",
  "script-and-regex.regex": "/^(?P<message>.*)/"
}

...to produce this diff. Also made a variant of it which matches lines to make sure that still works.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to If the Script-and-Regex linter captures no "line" text, treat the message as affecting the entire file.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
chad added inline comments.
src/lint/linter/ArcanistScriptAndRegexLinter.php
1

I feel like there should be something spinning here.

This revision is now accepted and ready to land.Jan 12 2016, 1:28 AM
epriestley edited edge metadata.
  • Remove example changes to .arclint now that we've all had a right jolly giggle.
This revision was automatically updated to reflect the committed changes.