Skip to content

Commit ccd6d84

Browse files
committed
Merge pull request #9 from codeclimate/gd-lint
Lint Engine itself
2 parents 16f7a9e + f52ab18 commit ccd6d84

File tree

4 files changed

+22
-14
lines changed

4 files changed

+22
-14
lines changed

.codeclimate.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,4 @@ engines:
33
enabled: true
44
ratings:
55
paths:
6-
- "bin/eslint"
6+
- "bin/eslint.js"

.eslintrc

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
{
2+
"env": {
3+
"node": true
4+
},
5+
"rules": {
6+
"strict": 0
7+
}
8+
}

Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,4 +7,4 @@ RUN npm install
77

88
COPY . /usr/src/app
99

10-
CMD ["/usr/src/app/bin/eslint"]
10+
CMD ["/usr/src/app/bin/eslint.js"]

bin/eslint renamed to bin/eslint.js

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ function buildIssueJson(message, path) {
88
// token)
99
var checkName = message.ruleId;
1010
if(message.fatal) {
11-
checkName = "fatal"
11+
checkName = "fatal";
1212
}
1313

1414
var issue = {
@@ -34,27 +34,27 @@ function buildIssueJson(message, path) {
3434
}
3535

3636
var options = {
37-
extensions: ['.js'], ignore: true, reset: false, useEslintrc: true
37+
extensions: [".js"], ignore: true, reset: false, useEslintrc: true
3838
};
39-
var ignores = []
39+
var ignores = [];
4040
if (process.env.ENGINE_CONFIG) {
4141
var engineConfig = JSON.parse(process.env.ENGINE_CONFIG);
42-
if (engineConfig['config']) {
43-
options['configFile'] = "/code/" + engineConfig['config'];
42+
if (engineConfig.config) {
43+
options.configFile = "/code/" + engineConfig.config;
4444
}
45-
if (engineConfig['exclude_paths']) {
46-
ignores = engineConfig['exclude_paths'];
45+
if (engineConfig.exclude_paths) {
46+
ignores = engineConfig.exclude_paths;
4747
}
48-
if (engineConfig['extensions']) {
49-
options['extensions'] = engineConfig['extensions'];
48+
if (engineConfig.extensions) {
49+
options.extensions = engineConfig.extensions;
5050
}
5151
}
5252
var cli = new CLIEngine(options);
5353
var report = cli.executeOnFiles(["/code"]);
54-
report.results.forEach(function(result, _, _) {
55-
var path = result.filePath.replace(/^\/code\//, '');
54+
report.results.forEach(function(result) {
55+
var path = result.filePath.replace(/^\/code\//, "");
5656
if (ignores.indexOf(path) === -1) {
57-
result.messages.forEach(function(message, _, _) {
57+
result.messages.forEach(function(message) {
5858
var issueJson = buildIssueJson(message, path);
5959
console.log(issueJson + "\u0000");
6060
});

0 commit comments

Comments
 (0)