HomePhabricator

Improve parsing of `csslint` output

Description

Improve parsing of csslint output

Summary: csslint returns an evidence field which contains the line of the "original text" which produced the linter message (see https://github.com/CSSLint/csslint/blob/5dd84b259b90c5ea8adbaf03d31e9df093cc3dcd/src/core/Reporter.js#L64). We can use this data instead of trying to achieve the same result using $this->getData($path).

Test Plan:

csslint (362×611 px, 29 KB)

Reviewers: chad, Blessed Reviewers, epriestley

Reviewed By: Blessed Reviewers, epriestley

Subscribers: Korvin, epriestley

Differential Revision: https://secure.phabricator.com/D11311

Details

Provenance
joshuaspenceAuthored on
joshuaspencePushed on Jan 11 2015, 7:48 PM
Reviewer
Blessed Reviewers
Differential Revision
D11311: Improve parsing of `csslint` output
Parents
rARC7b383632dde1: Minor tidying of `ArcanistCSSLintLinter`
Branches
Unknown
Tags
Unknown