Page MenuHomePhabricator

Fix ArcanistCSSLintLinter issue for messages without line number
ClosedPublic

Authored by dereckson on Nov 17 2015, 5:31 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Mar 29, 2:03 PM
Unknown Object (File)
Mar 8 2024, 12:48 PM
Unknown Object (File)
Mar 8 2024, 12:48 PM
Unknown Object (File)
Mar 5 2024, 7:03 PM
Unknown Object (File)
Mar 5 2024, 2:24 AM
Unknown Object (File)
Feb 16 2024, 3:33 PM
Unknown Object (File)
Feb 15 2024, 7:34 PM
Unknown Object (File)
Feb 11 2024, 10:00 AM
Subscribers

Details

Summary

csslint offers some diagnostics related to all the document without any
relevant line number.

In such case, arc lint failed, as setLine expects null or an integer,
but not an empty string.

The 'char' and 'line' attributes in XML report are now optional.

Fixes T9804.

Test Plan

test case to repro the issue added

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dereckson retitled this revision from to Fix ArcanistCSSLintLinter issue for messages without line number.
dereckson updated this object.
dereckson edited the test plan for this revision. (Show Details)
dereckson added reviewers: epriestley, joshuaspence.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Nov 17 2015, 3:13 PM
This revision was automatically updated to reflect the committed changes.