Migrated from jshint -> eslint #1930
Conversation
gulpfile.babel.js
Outdated
| @@ -149,18 +149,6 @@ gulp.task('copy:normalize', () => | |||
| .pipe(gulp.dest(`${dirs.dist}/css`)) | |||
| ); | |||
|
|
|||
| gulp.task('lint:js', () => | |||
This comment was marked as abuse.
This comment was marked as abuse.
Sorry, something went wrong.
gulpfile.babel.js
Outdated
| @@ -175,7 +163,7 @@ gulp.task('archive', (done) => { | |||
|
|
|||
| gulp.task('build', (done) => { | |||
| runSequence( | |||
| ['clean', 'lint:js'], | |||
This comment was marked as abuse.
This comment was marked as abuse.
Sorry, something went wrong.
| @@ -1,24 +1,23 @@ | |||
| // Avoid `console` errors in browsers that lack a console. | |||
| (function() { | |||
This comment was marked as abuse.
This comment was marked as abuse.
Sorry, something went wrong.
| @@ -1,24 +1,23 @@ | |||
| // Avoid `console` errors in browsers that lack a console. | |||
| (function() { | |||
This comment was marked as abuse.
This comment was marked as abuse.
Sorry, something went wrong.
.eslintrc
Outdated
| @@ -1,3 +1,12 @@ | |||
| { | |||
| "extends": "eslint:recommended" | |||
This comment was marked as abuse.
This comment was marked as abuse.
Sorry, something went wrong.
| // should be included at the end | ||
| 'css/main.css', |
This comment was marked as abuse.
This comment was marked as abuse.
Sorry, something went wrong.
|
@amilajack I finaly had time to take a look at the changes and had some issues. There's also a merge conflict since we added modernizr 3.0. |
|
@roblarsen: you can always fix the issues and push to @amilajack's fork if they take a long time to respond :) |
|
@XhmikosR it's been a while so I'm not actually expecting this to get updated. At some point I was going to revisit. |
|
Closing this and we'll revisit in a more orderly fashion. |
Closes #1913