Page MenuHomePhabricator
Differential D14165

Improve validation of 'name' and 'code' parameters for lint messages
ClosedPublic
Actions

Authored by epriestley on Sep 25 2015, 5:02 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 24, 10:31 PM
Unknown Object (File)
Mon, Apr 22, 8:48 PM
Unknown Object (File)
Fri, Apr 19, 3:38 PM
Unknown Object (File)
Sun, Mar 31, 6:08 PM
Unknown Object (File)
Mar 15 2024, 4:41 PM
Unknown Object (File)
Mar 15 2024, 4:11 PM
Unknown Object (File)
Mar 15 2024, 3:07 PM
Unknown Object (File)
Mar 15 2024, 2:11 PM
View All Files
Subscribers
aik099

Details

Reviewers
chad
Maniphest Tasks
T9145: Exception on arc diff with PHPCS linter
T9316: Linters must include Name fields
Commits
rARCac28f951d69c: (stable) Promote 2015 Week 39
rARCe8a0ebaeffaa: Improve validation of 'name' and 'code' parameters for lint messages
Summary

Ref T9145. Fixes T9316. We now require "name" and "code" has a maximum length (currently, this is 32, but the next diff will raise it to 128).

Test Plan
  • Installed PHPCS.
  • Hit both the "name" and "code" issues.
  • Applied this patch.
  • Got better errors sooner.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 34222.Sep 25 2015, 5:02 PM
epriestley retitled this revision from to Improve validation of 'name' and 'code' parameters for lint messages.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added tasks: T9316: Linters must include Name fields, T9145: Exception on arc diff with PHPCS linter.
epriestley mentioned this in T9316: Linters must include Name fields.Sep 25 2015, 5:05 PM
aik099 added a subscriber: aik099.Sep 25 2015, 5:15 PM
chad accepted this revision.Sep 25 2015, 6:11 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Sep 25 2015, 6:11 PM
Closed by commit rARCe8a0ebaeffaa: Improve validation of 'name' and 'code' parameters for lint messages (authored by epriestley, committed by epriestley). · Explain WhySep 25 2015, 6:16 PM
This revision was automatically updated to reflect the committed changes.
tedkornish mentioned this in D14182: Add a name to Golang ArcanistLintMessages.Sep 28 2015, 10:00 PM
vm mentioned this in D14225: Set name on more linters.Oct 2 2015, 3:57 PM
epriestley mentioned this in rARC1b4a3e0c5eb9: Set path on more linters.Oct 2 2015, 3:58 PM
dereckson mentioned this in D11657: Add a ShellCheck linter.Mar 14 2017, 12:01 PM
epriestley mentioned this in T12981: Explicitly limit lint and unit test names to 255 characters.Sep 12 2017, 7:08 PM

Revision Contents
Changeset List

PathSize
src/
lint/
17 lines
engine/
15 lines
linter/
20 lines

Diff 34234

View Options

src/lint/ArcanistLintMessage.php

Loading...
View Options

src/lint/engine/ArcanistLintEngine.php

Loading...
View Options

src/lint/linter/ArcanistPhpcsLinter.php

Loading...
Phabricator · Built by Phacility