Bug 508847 - [CSS] Update to CSSLint 1.0.4
Summary: [CSS] Update to CSSLint 1.0.4
Status: RESOLVED FIXED
Alias: None
Product: Orion (Archived)
Classification: ECD
Component: JS Tools (show other bugs)
Version: 13.0   Edit
Hardware: PC Windows NT
: P3 normal (vote)
Target Milestone: 15.0   Edit
Assignee: Curtis Windatt CLA
QA Contact:
URL:
Whiteboard:
Keywords: noteworthy
Depends on:
Blocks: 458306 512330 512586 513250
  Show dependency tree
 
Reported: 2016-12-07 11:39 EST by Curtis Windatt CLA
Modified: 2017-03-20 16:35 EDT (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Curtis Windatt CLA 2016-12-07 11:39:26 EST
It has been years since we updates CSSLint and there are many improvements we could take advantage of.
Comment 1 Michael Rennie CLA 2016-12-12 23:56:56 EST
Here's the diff from current master to what we are consuming:

https://github.com/CSSLint/csslint/compare/v0.10.0...master

there is a lot of improvements to the project shape / package.json / build / code formatting, but not much in the way of tooling improvement.
Comment 2 Curtis Windatt CLA 2017-02-16 16:34:57 EST
Parser-lib has major changes to its known properties to support CSS3 and SVG.

The built library increased by 1500 LoC (~10800 long now).
Comment 3 Curtis Windatt CLA 2017-02-22 15:23:54 EST
Another change is the addition of ignore directives that disable the linting for parts of your file.

https://github.com/CSSLint/csslint/wiki/Ignoring-parts-of-CSS-during-linting
Comment 4 Curtis Windatt CLA 2017-02-23 15:30:02 EST
Confirmed that the 1.0.4 dist is compatible with our current code. We will have to modify it to:
a) export colours
b) allow NLS of messages
c) possible line ending problems
d) export the ast
e) don't fail on no source
f) possible tokenizer changes?
g) change severity to INFO

Opened a CQ:
https://dev.eclipse.org/ipzilla/show_bug.cgi?id=12779
Comment 5 Curtis Windatt CLA 2017-03-01 17:41:31 EST
http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/?h=cwindatt%2FBug508847_CSSLint104

CQ was approved and I have redone our customizations. Orion git client was having a problem resetting the active branch, so for now I put the changes into a branch. As far as I can tell, the new version is ready to go live.

> a) export colours
Done
> b) allow NLS of messages
Done, nls existing, added 2, changed text of one
> c) possible line ending problems
Done
> d) export the ast
Done, but there are new node types we should handle in the future
> e) don't fail on no source
Done
> f) possible tokenizer changes?
Done see (d)
> g) change severity to INFO
Done

h) Update tests to not have a syntax error (undetectable in previous csslint version)
Done
i) SVG properties
Already handled in this version
Comment 6 Curtis Windatt CLA 2017-03-07 12:56:02 EST
As rules were added and updated, I will need to review our settings in webtoolsplugin.js and the matching config properties in cssvalidator.js.

Also after the update we will try to improve the messages and severities. See Bug 513250.