Skip to content

Commit a89faab

Browse files
authored
fix: allow ignore files to be read (#100)
* fix: allow ignore files to be read * fix: skip general messages reported without line numbers
1 parent 21d323c commit a89faab

File tree

1 file changed

+12
-5
lines changed

1 file changed

+12
-5
lines changed

src/tools/js/eslint.js

Lines changed: 12 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,8 @@ module.exports = (file, text, apply = false) => {
2828
const options = {
2929
baseConfig: require(eslintConfig),
3030
fix: apply,
31-
ignore: false,
31+
ignore: true,
32+
reportUnusedDisableDirectives: true,
3233
}
3334
try {
3435
const engine = new eslint.CLIEngine({
@@ -72,10 +73,16 @@ module.exports = (file, text, apply = false) => {
7273
if (message.ruleId) {
7374
rule = `(${message.ruleId})`
7475
}
75-
response.messages.push({
76-
checker: 'eslint',
77-
message: `Line ${message.line}: ${message.message} ${rule}`,
78-
})
76+
if (message.line) {
77+
response.messages.push({
78+
checker: 'eslint',
79+
message: `Line ${message.line}: ${message.message} ${rule}`,
80+
})
81+
} else {
82+
log.debug(
83+
`No line number, assuming general message: ${message.message}`
84+
)
85+
}
7986
}
8087
} catch (error) {
8188
log.error(`ESLint format failed with error:\n${error}`)

0 commit comments

Comments
 (0)