Skip to content

Commit

Permalink
Use ES6 linter for build-system (#11357)
Browse files Browse the repository at this point in the history
  • Loading branch information
rsimha authored Sep 20, 2017
1 parent 1bf0864 commit 5b138aa
Show file tree
Hide file tree
Showing 4 changed files with 118 additions and 66 deletions.
8 changes: 0 additions & 8 deletions build-system/.eslintrc

This file was deleted.

7 changes: 1 addition & 6 deletions build-system/tasks/lint.js
Original file line number Diff line number Diff line change
Expand Up @@ -34,10 +34,6 @@ var options = {
rulePaths: ['build-system/eslint-rules/'],
plugins: ['eslint-plugin-google-camelcase'],
};
var buildSystemOptions = {
fix: false,
plugins: ['eslint-plugin-google-camelcase'],
};

/**
* On travis, we'll start by linting just the build-system files that are being
Expand Down Expand Up @@ -124,12 +120,11 @@ function runLinter(path, stream, options) {
function lint() {
if (argv.fix) {
options.fix = true;
buildSystemOptions.fix = true;
}
if (argv.build_system) {
var stream =
initializeStream(getBuildSystemFiles(), { base: 'build-system' });
return runLinter('./build-system/', stream, buildSystemOptions);
return runLinter('./build-system/', stream, options);
} else {
var stream = initializeStream(config.lintGlobs, {});
return runLinter('.', stream, options);
Expand Down
1 change: 0 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,6 @@
"cssnano": "3.3.2",
"del": "2.2.0",
"eslint": "3.18.0",
"eslint-config-es5": "0.5.0",
"eslint-plugin-google-camelcase": "0.0.2",
"esprima": "3.1.3",
"express": "4.14.0",
Expand Down
Loading

0 comments on commit 5b138aa

Please sign in to comment.