Page MenuHomePhabricator
Differential D8965

Modernize `ArcanistJSHintLinter`.
ClosedPublic
Actions

Authored by joshuaspence on May 3 2014, 6:22 AM.
Tags
None
Referenced Files
F15425811: D8965.id21331.diff
Sun, Mar 23, 6:15 AM
F15416798: D8965.diff
Thu, Mar 20, 1:33 PM
F15375434: D8965.diff
Wed, Mar 12, 11:12 PM
Unknown Object (File)
Jan 25 2025, 8:30 PM
Unknown Object (File)
Jan 21 2025, 9:00 AM
Unknown Object (File)
Jan 19 2025, 5:08 PM
Unknown Object (File)
Jan 18 2025, 2:05 AM
Unknown Object (File)
Dec 31 2024, 9:43 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
rARCa7327ca0e943: Modernize `ArcanistJSHintLinter`.
Summary

Modernize ArcanistJSHintLinter by extending from ArcanistExternalLinter instead of ArcanistLinter.

Test Plan

Wrote and executed unit tests.

Diff Detail

Repository
rARC Arcanist
Branch
jshint
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 216
Build 216: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence updated this revision to Diff 21265.May 3 2014, 6:22 AM
joshuaspence retitled this revision from to Modernize `ArcanistJSHintLinter`..
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMay 3 2014, 6:22 AM
Herald added subscribers: Korvin, epriestley. · View Herald Transcript
Harbormaster completed remote builds in B216: Diff 21265.May 3 2014, 6:23 AM
joshuaspence added a comment.May 4 2014, 11:52 AM

Actually this overlaps a fair bit with D6810.

epriestley accepted this revision.May 5 2014, 9:22 PM
epriestley edited edge metadata.

Thanks!

This revision is now accepted and ready to land.May 5 2014, 9:22 PM
epriestley added inline comments.May 5 2014, 9:22 PM
src/lint/linter/__tests__/jshint/dot-notation.lint-test
2–3

My local copy doesn't raise these, but we can sort that out once the version stuff is in.

epriestley closed this revision.May 5 2014, 9:23 PM
epriestley updated this revision to Diff 21331.

Closed by commit rARCa7327ca0e943 (authored by @joshuaspence, committed by @epriestley).

Revision Contents
Changeset List

Diff 21265

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/lint/linter/ArcanistJSHintLinter.php

Loading...
View Options

src/lint/linter/__tests__/ArcanistJSHintLinterTestCase.php

Loading...
View Options

src/lint/linter/__tests__/jshint/dot-notation.lint-test

Loading...
View Options

src/lint/linter/__tests__/jshint/expected-conditional.lint-test

Loading...
View Options

src/lint/linter/__tests__/jshint/missing-semicolon.lint-test

Loading...
View Options

src/lint/linter/__tests__/jshint/unnecessary-semicolon.lint-test

Loading...
View Options

src/lint/linter/reporter.js

Loading...
Phabricator · Built by Phacility