This repository has been archived by the owner on Aug 19, 2022. It is now read-only.
Infra: Remove prettier trailingComma #932
Comments
ryan-roemer
added a commit
that referenced
this issue
Oct 25, 2017
ryan-roemer
added a commit
that referenced
this issue
Oct 25, 2017
Trailing commas add no "value" for correctness, and their absence is "always correct" for thing like older version of Node. Now that we actually lint all our JS files in Radium, it is easier to just remove trailing commas in lint than have different lint for transpiled src vs. "real node" root *.js files like webpack configs, etc.
jaredbrookswhite
pushed a commit
to jaredbrookswhite/radium
that referenced
this issue
Aug 22, 2018
…abs#940) Trailing commas add no "value" for correctness, and their absence is "always correct" for thing like older version of Node. Now that we actually lint all our JS files in Radium, it is easier to just remove trailing commas in lint than have different lint for transpiled src vs. "real node" root *.js files like webpack configs, etc.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
And then run
yarn run fixlintThe text was updated successfully, but these errors were encountered: