Skip to content

Coding Standards: JS: Pin Typescript Version#306

Merged
n7studios merged 1 commit into
mainfrom
js-coding-standards-pin-typescript-version
Jul 10, 2026
Merged

Coding Standards: JS: Pin Typescript Version#306
n7studios merged 1 commit into
mainfrom
js-coding-standards-pin-typescript-version

Conversation

@n7studios

Copy link
Copy Markdown
Contributor

Summary

Fixes JS linting due to Typescript 7's release yesterday, which some npm packages don't yet support:

wp-scripts lint-js --max-warnings=0
Oops! Something went wrong! :(
ESLint: 8.57.1
TypeError: Failed to load plugin '@typescript-eslint' declared in '.eslintrc.js » plugin:@wordpress/recommended': Cannot read properties of undefined (reading 'Intrinsic')
Referenced from: /Users/tim/Local Sites/_plugins/convertkit-wpforms/node_modules/@wordpress/eslint-plugin/index.js
    at Object.<anonymous> (/Users/tim/Local Sites/_plugins/convertkit-wpforms/node_modules/ts-api-utils/lib/index.cjs:779:57)
    at Module._compile (node:internal/modules/cjs/loader:1688:14)
    at Object..js (node:internal/modules/cjs/loader:1820:10)
    at Module.load (node:internal/modules/cjs/loader:1423:32)
    at Function._load (node:internal/modules/cjs/loader:1246:12)
    at TracingChannel.traceSync (node:diagnostics_channel:322:14)
    at wrapModuleLoad (node:internal/modules/cjs/loader:235:24)
    at Module.require (node:internal/modules/cjs/loader:1445:12)
    at require (node:internal/modules/helpers:135:16)
    at Object.<anonymous> (/Users/tim/Local Sites/_plugins/convertkit-wpforms/node_modules/@typescript-eslint/type-utils/dist/containsAllTypesByName.js:27:30)
Screenshot 2026-07-09 at 11 00 08

Testing

Existing tests pass.

Checklist

@n7studios n7studios self-assigned this Jul 9, 2026
@n7studios n7studios added the bug label Jul 9, 2026
@n7studios n7studios requested review from a team, ciccio-kit and noelherrick and removed request for a team July 9, 2026 04:44
@n7studios n7studios marked this pull request as ready for review July 9, 2026 04:44
@n7studios n7studios merged commit 479c031 into main Jul 10, 2026
110 of 112 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants