Skip to content

fix(theme): replace jshint with eslint to resolve minimatch vulnerability#1943

Merged
johanib merged 1 commit intomainfrom
feature/update-js
Mar 17, 2026
Merged

fix(theme): replace jshint with eslint to resolve minimatch vulnerability#1943
johanib merged 1 commit intomainfrom
feature/update-js

Conversation

@johanib
Copy link
Copy Markdown
Contributor

@johanib johanib commented Mar 17, 2026

jshint@2.13.6 requires minimatch@~3.0.2 which excludes all non-vulnerable versions (>=3.1.4), making a Dependabot security update impossible.

  • Replace jshint with eslint@^9.0 + globals@^16.0 in devDependencies
  • Add eslint.config.js with eslint:recommended, browser globals, and caughtErrors:none for no-unused-vars
  • Fix no-case-declarations in keyboardBehaviour.js
  • Suppress no-self-assign for intentional screen reader hack in showOrHideNoResultsSection.js

@johanib johanib force-pushed the feature/update-js branch from 8ebd4d6 to a2d1fa5 Compare March 17, 2026 08:57
@johanib johanib requested a review from kayjoosten March 17, 2026 08:57
Comment thread theme/base/javascripts/wayf/utility/sortAndReindex.js
…lity

jshint@2.13.6 requires minimatch@~3.0.2 which excludes all non-vulnerable
versions (>=3.1.4), making a Dependabot security update impossible.
- Replace jshint with eslint@^9.0 + globals@^16.0 in devDependencies
- Add eslint.config.js with eslint:recommended, browser globals, and
  caughtErrors:none for no-unused-vars
- Fix no-case-declarations in keyboardBehaviour.js
- Suppress no-self-assign for intentional screen reader hack in
  showOrHideNoResultsSection.js
@johanib johanib force-pushed the feature/update-js branch from a2d1fa5 to 88f8537 Compare March 17, 2026 09:29
@johanib johanib merged commit 6c73477 into main Mar 17, 2026
2 checks passed
@johanib johanib deleted the feature/update-js branch March 17, 2026 10:00
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.

2 participants