disable lint as part of build process

[bdefore]

there are a bunch of linting errors that break npm run build right now

[oliviertassinari]

You can close this issue/PR. The CI tool is linting the code and everything is fine.
I'm preaty sure this fail for you because you have an other .eslintrc higher in the directory hierarchy.

[bdefore]

@oliviertassinari Is the CI linting though? I see the most recent running tests but not linting: https://travis-ci.org/callemall/material-ui/builds/84334520

[oliviertassinari]

@bdefore Yes he does #979. The prepublish script is doing it.

prepublish: Run BEFORE the package is published. (Also run on local npm install without any arguments.)

(https://docs.npmjs.com/misc/scripts)

Plus, the failing lint rule that you described isn't enforced by the .eslintrc of this project.

[chrismcv]

any thoughts on disabling linting on the docs project - when I'm coding I find it really slows things down when you need to go back and add dangling comma's and other incidentals that don't matter at dev time...

[oliviertassinari]

@chrismcv Are you talking about the webpack-eslint pluggin? Then, I would suggest to enable the fix option so that eslint fix dangling comma's for you.

[oliviertassinari]

@chrismcv Since #1887 solve it, can you close this PR?

[chrismcv]

@bdefore - I can't close, can you?

[oliviertassinari]

@chrismcv Sorry, @bdefore