Skip to content

AILogAnalyzer: fix ESLint problems - #344

Open
j-w9 wants to merge 3 commits into
ArduPilot:mainfrom
j-w9:lint-AILogAnalyzer
Open

j-w9 wants to merge 3 commits into
ArduPilot:mainfrom
j-w9:lint-AILogAnalyzer

Conversation

@j-w9

@j-w9 j-w9 commented Oct 11, 2026

Copy link
Copy Markdown
Contributor

Fixes ESLint problems in AILogAnalyzer and removes the fixed entries from eslint-suppressions.json. None of the changes alter behaviour.

  • Scope const declarations in case blocks. Wrap the case in braces so its const does not leak into the other cases of the switch.
  • Drop unused catch bindings. Use catch without a binding where the error is not used.

One commit per kind of fix; npm run lint passes.

j-w9 added 3 commits October 13, 2026 22:30
Wrap the case in braces so its const does not leak into the other cases
of the switch.
Use catch without a binding where the error is not used.
@j-w9
j-w9 force-pushed the lint-AILogAnalyzer branch from 5efddd3 to 6ed85ae Compare October 13, 2026 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant