search

jsdom / cssstyle

A Node.js implementation of the CSS Object Model CSSStyleDeclaration interface
MIT License
109 stars 70 forks source link

fix: remove the extra comma that would cause error in some old apps #82

Closed loking closed 5 years ago

loking commented 5 years ago

To fix #81

fatfisz commented 5 years ago

~Hi, there's a problem with that branch: we have some warnings in the build because after running prettier again the comma would be restored. Could you first add --max-warnings 0 after "eslint" in the lint and lint:fix commands?~ I've done this, this is on master. Please start from the current master.

After that you could change trailingComma in .eslintrc.js to "es5" and run npm run lint:fix - this will catch more trailing commas.