Page MenuHomePhabricator

Warn when accessing deprecated lint config
ClosedPublic

Authored by epriestley on May 11 2014, 11:42 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 15, 7:08 AM
Unknown Object (File)
Wed, Nov 27, 3:25 PM
Unknown Object (File)
Sat, Nov 23, 10:15 PM
Unknown Object (File)
Nov 22 2024, 10:30 PM
Unknown Object (File)
Nov 19 2024, 7:00 AM
Unknown Object (File)
Nov 17 2024, 1:01 PM
Unknown Object (File)
Nov 11 2024, 9:54 PM
Unknown Object (File)
Nov 5 2024, 7:08 PM
Subscribers

Details

Reviewers
joshuaspence
btrahan
chad
Maniphest Tasks
Restricted Maniphest Task
Commits
rARC7870e7f2e46d: Warn when accessing deprecated lint config
Summary

Ref T2039. Addresses two issues:

  • Issues a warning for use of config which is deprecated by .arclint.
  • We no longer require an engine to be present for these linters, so arc linters doesn't fatal if they aren't configured.
Test Plan
  • Ran arc linters in a repo with no JSHint.
  • Ran arc linters in a repo with junk in .arcconfig and got a warning when it was read. Verified it still took effect.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Warn when accessing deprecated lint config.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: joshuaspence, btrahan.
epriestley added a task: Restricted Maniphest Task.
joshuaspence edited edge metadata.

Looks good to me.

This revision is now accepted and ready to land.May 11 2014, 11:45 PM
epriestley updated this revision to Diff 21513.

Closed by commit rARC7870e7f2e46d (authored by @epriestley).