Paths

Table of Contentst

Differential D20541

Use the checkstyle reporter with JSHint
Needs ReviewPublic
Actions

Authored by joshuaspence on May 22 2019, 12:59 AM.

Tags

None

Referenced Files

	F15497238: D20541.id48986.diff
	Sun, Apr 13, 10:06 AM

	F15492715: D20541.id.diff
	Sat, Apr 12, 5:35 PM

	F15491198: D20541.diff
	Sat, Apr 12, 12:06 AM

	F15490208: D20541.diff
	Fri, Apr 11, 3:15 PM

	F15470827: D20541.id.diff
	Sat, Apr 5, 1:43 AM

	F15465560: D20541.diff
	Wed, Apr 2, 8:36 PM

	F15439054: D20541.id.diff
	Wed, Mar 26, 5:28 AM

	Unknown Object (File)
	Feb 11 2025, 4:40 AM

View All Files

Subscribers

Korvin

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Summary

Use the built-in checkstyle reporter instead of our custom reporter.js file for ArcanistJSHintLinter. --reporter=checkstyle has been available since at least 2012.

Test Plan

Ran unit tests.

Diff Detail

Repository

rARC Arcanist

Branch

lint3

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 22862
Build 31365: Run Core Tests
Build 31364: arc lint + arc unit

Event Timeline

joshuaspence created this revision.May 22 2019, 12:59 AM

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMay 22 2019, 12:59 AM

Harbormaster completed remote builds in B22862: Diff 48986.May 22 2019, 1:00 AM

joshuaspence requested review of this revision.May 22 2019, 1:00 AM

Herald added a subscriber: Korvin. · View Herald TranscriptMay 22 2019, 1:00 AM

Revision Contents
Changeset List

Path

Size

Coverage (All)

Coverage (Touched)

src/

lint/

linter/

ArcanistJSHintLinter.php

36 lines

52%

reporter.js

Diff 48986

View Options

src/lint/linter/ArcanistJSHintLinter.php