CSSLint / csslint

Automated linting of Cascading Stylesheets
http://csslint.net
Other
4.77k stars 483 forks source link

Make npm usable with webapp #673

Closed orionlee closed 7 years ago

orionlee commented 8 years ago

For csslint's npm package , could it include core (i.e., dist/csslint.js) as well?

Use case: In my webapp, I use npm to bring in dependent libraries, so that I can include them in following manner:

  <script type="text/javascript" src="node_modules/csslint/dist/csslint.js"></script>

However, csslint npm does not provide the dist/csslint.js. Could it be included? Thanks.

XhmikosR commented 7 years ago

I don't see any downside.

Please make a PR by changing https://github.com/CSSLint/csslint/blob/master/package.json#L44 to dist/ and CC me.

mkarajohn commented 7 years ago

@XhmikosR When can we expect an update to the npm package? As of the moment of writing this, the csslint.js file is still missing in a fresh installation.

XhmikosR commented 7 years ago

@mkarajohn: done.