Skip to content

Commit 5cb7096

Browse files
review comment fix
1 parent 0f0692d commit 5cb7096

1 file changed

Lines changed: 9 additions & 9 deletions

File tree

  • packages/code-analyzer-eslint-engine/src

packages/code-analyzer-eslint-engine/src/engine.ts

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -296,15 +296,15 @@ function toViolation(file: string, resultMsg: Linter.LintMessage,
296296
primaryLocationIndex: 0
297297
};
298298

299-
if (lineStartOffsets) {
300-
if (includeFixes && resultMsg.fix) {
301-
violation.fixes = [convertEslintFix(file, resultMsg.fix, lineStartOffsets)];
302-
}
303-
304-
if (includeSuggestions && resultMsg.suggestions?.length) {
305-
violation.suggestions = resultMsg.suggestions.map(s =>
306-
convertEslintSuggestion(file, s, lineStartOffsets));
307-
}
299+
if (!lineStartOffsets) {
300+
return violation;
301+
}
302+
if (includeFixes && resultMsg.fix) {
303+
violation.fixes = [convertEslintFix(file, resultMsg.fix, lineStartOffsets)];
304+
}
305+
if (includeSuggestions && resultMsg.suggestions?.length) {
306+
violation.suggestions = resultMsg.suggestions.map(s =>
307+
convertEslintSuggestion(file, s, lineStartOffsets));
308308
}
309309

310310
return violation;

0 commit comments

Comments
 (0)