Skip to content

[MAINTENANCE] Resolve Codacy no-underscore-dangle issues - #2162

Merged
sebastian-meyer merged 2 commits into
kitodo:mainfrom
stweil:fix/codacy-no-underscore-dangle
Sep 25, 2026
Merged

sebastian-meyer merged 2 commits into
kitodo:mainfrom
stweil:fix/codacy-no-underscore-dangle

Conversation

@stweil

@stweil stweil commented Sep 21, 2026

Copy link
Copy Markdown
Member

Codacy's ESLint8 no-underscore-dangle pattern reports 335 issues, all in vendored third-party libraries (OpenLayers, jQuery UI, pdfkit, shaka-player, gridstack, model-viewer, ...) and the webpack-generated DlfMediaVendor.js bundle. Renaming private members in vendored code is not an option, so stop linting those files and disable the rule for the rest of the codebase, where jQuery UI's private-method convention (_method, _super) is used by first-party components.

Assisted-by: OpenCode / qwen3.8-27b (Alibaba Cloud)

Codacy's ESLint8 no-underscore-dangle pattern reports 335 issues, all in
vendored third-party libraries (OpenLayers, jQuery UI, pdfkit, shaka-player,
gridstack, model-viewer, ...) and the webpack-generated DlfMediaVendor.js
bundle. Renaming private members in vendored code is not an option, so
stop linting those files and disable the rule for the rest of the
codebase, where jQuery UI's private-method convention (_method, _super)
is used by first-party components.

Assisted-by: OpenCode / qwen3.8-27b (Alibaba Cloud)
Signed-off-by: Stefan Weil <stweil@users.noreply.github.com>
@sebastian-meyer sebastian-meyer added the ⟳ maintenance A task to keep the code up-to-date and manageable. label Sep 25, 2026
@sebastian-meyer
sebastian-meyer merged commit 875b458 into kitodo:main Sep 25, 2026
10 checks passed
@stweil
stweil deleted the fix/codacy-no-underscore-dangle branch September 25, 2026 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⟳ maintenance A task to keep the code up-to-date and manageable.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants