-
-
Notifications
You must be signed in to change notification settings - Fork 308
chore: bump TypeScript from 5 to 7, and add TypeScript 6 side by side #9518
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
32eeb22
74a0e5f
9cecb2c
58713b9
c95e510
4d89424
50c26bd
48c7fd1
75d3e9a
b4eaddf
941e4a3
a68cc71
eb59804
7c8fef1
f9ffbcf
e304fb8
d49f800
7d6c717
84400a2
15bc787
f1a3bfd
c925b0f
399206b
f2b52e1
8dc33f1
3bdec3d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,15 @@ | ||
| diff --git a/package.json b/package.json | ||
| index 202f5f7353c0ba60756102dc03ca58cdcf4333c5..17d4e72004937d35c9b564584215969a36571c93 100644 | ||
| --- a/package.json | ||
| +++ b/package.json | ||
| @@ -21,10 +21,6 @@ | ||
| }, | ||
| "main": "./lib/typescript.js", | ||
| "typings": "./lib/typescript.d.ts", | ||
| - "bin": { | ||
| - "tsc": "./bin/tsc", | ||
| - "tsserver": "./bin/tsserver" | ||
| - }, | ||
| "engines": { | ||
| "node": ">=14.17" | ||
| }, |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -207,6 +207,7 @@ module.exports = { | |
| module: 'CommonJS', | ||
| moduleResolution: 'Node', | ||
| verbatimModuleSyntax: false, | ||
| ignoreDeprecations: '6.0', | ||
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
|
||
| }, | ||
| }, | ||
| ], | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -71,14 +71,15 @@ | |
| "@metamask/eth-hd-keyring": "^15.0.0", | ||
| "@metamask/providers": "^22.1.0", | ||
| "@types/jest": "^30.0.0", | ||
| "@typescript/native": "npm:typescript@^7.0.2", | ||
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
| "deepmerge": "^4.2.2", | ||
| "jest": "^30.4.2", | ||
| "rimraf": "^5.0.5", | ||
| "ts-jest": "^29.4.11", | ||
| "tsx": "^4.20.5", | ||
| "typedoc": "^0.25.13", | ||
| "typedoc-plugin-missing-exports": "^2.0.0", | ||
| "typescript": "~5.3.3", | ||
| "typescript": "npm:@typescript/typescript6@^6.0.2", | ||
| "webextension-polyfill": "^0.12.0" | ||
| }, | ||
| "peerDependencies": { | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -22,7 +22,7 @@ export function getLocalEntropyWallets( | |
| ): AccountWalletEntropyObject[] { | ||
| return Object.values(context.controller.state.accountTree.wallets).filter( | ||
| (wallet) => wallet.type === AccountWalletType.Entropy, | ||
| ) as AccountWalletEntropyObject[]; | ||
| ); | ||
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. TypeScript 6+ has better support for |
||
| } | ||
|
|
||
| /** | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,17 +1,28 @@ | ||
| { | ||
| "extends": "../../tsconfig.packages.build.json", | ||
| "compilerOptions": { | ||
| "baseUrl": "./", | ||
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
|
||
| "outDir": "./dist", | ||
| "rootDir": "./src" | ||
| }, | ||
| "references": [ | ||
| { "path": "../accounts-controller/tsconfig.build.json" }, | ||
| { "path": "../base-controller/tsconfig.build.json" }, | ||
| { "path": "../keyring-controller/tsconfig.build.json" }, | ||
| { "path": "../messenger/tsconfig.build.json" }, | ||
| { "path": "../multichain-account-service/tsconfig.build.json" }, | ||
| { "path": "../profile-sync-controller/tsconfig.build.json" } | ||
| { | ||
| "path": "../accounts-controller/tsconfig.build.json" | ||
| }, | ||
| { | ||
| "path": "../base-controller/tsconfig.build.json" | ||
| }, | ||
| { | ||
| "path": "../keyring-controller/tsconfig.build.json" | ||
| }, | ||
| { | ||
| "path": "../messenger/tsconfig.build.json" | ||
| }, | ||
| { | ||
| "path": "../multichain-account-service/tsconfig.build.json" | ||
| }, | ||
| { | ||
| "path": "../profile-sync-controller/tsconfig.build.json" | ||
| } | ||
| ], | ||
| "include": ["../../types", "./src"], | ||
| "exclude": ["**/*.test.ts", "**/jest.config.ts", "./src/state/tests"] | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,14 +1,19 @@ | ||
| { | ||
| "extends": "../../tsconfig.packages.build.json", | ||
| "compilerOptions": { | ||
| "baseUrl": "./", | ||
| "outDir": "./dist", | ||
| "rootDir": "./src" | ||
| }, | ||
| "references": [ | ||
| { "path": "../base-controller/tsconfig.build.json" }, | ||
| { "path": "../controller-utils/tsconfig.build.json" }, | ||
| { "path": "../messenger/tsconfig.build.json" } | ||
| { | ||
| "path": "../base-controller/tsconfig.build.json" | ||
| }, | ||
| { | ||
| "path": "../controller-utils/tsconfig.build.json" | ||
| }, | ||
| { | ||
| "path": "../messenger/tsconfig.build.json" | ||
| } | ||
| ], | ||
| "include": ["../../types", "./src"] | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,12 +1,15 @@ | ||
| { | ||
| "extends": "../../tsconfig.packages.json", | ||
| "compilerOptions": { | ||
| "baseUrl": "./" | ||
| }, | ||
| "references": [ | ||
| { "path": "../base-controller" }, | ||
| { "path": "../controller-utils" }, | ||
| { "path": "../messenger" } | ||
| { | ||
| "path": "../base-controller" | ||
| }, | ||
| { | ||
| "path": "../controller-utils" | ||
| }, | ||
| { | ||
| "path": "../messenger" | ||
| } | ||
| ], | ||
| "include": ["../../types", "./src"] | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,13 +1,16 @@ | ||
| { | ||
| "extends": "../../tsconfig.packages.build.json", | ||
| "compilerOptions": { | ||
| "baseUrl": "./", | ||
| "outDir": "./dist", | ||
| "rootDir": "./src" | ||
| }, | ||
| "references": [ | ||
| { "path": "../base-controller/tsconfig.build.json" }, | ||
| { "path": "../messenger/tsconfig.build.json" } | ||
| { | ||
| "path": "../base-controller/tsconfig.build.json" | ||
| }, | ||
| { | ||
| "path": "../messenger/tsconfig.build.json" | ||
| } | ||
| ], | ||
| "include": ["../../types", "./src"] | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,8 +1,12 @@ | ||
| { | ||
| "extends": "../../tsconfig.packages.json", | ||
| "compilerOptions": { | ||
| "baseUrl": "./" | ||
| }, | ||
| "references": [{ "path": "../base-controller" }, { "path": "../messenger" }], | ||
| "references": [ | ||
| { | ||
| "path": "../base-controller" | ||
| }, | ||
| { | ||
| "path": "../messenger" | ||
| } | ||
| ], | ||
| "include": ["../../types", "./src"] | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,14 +1,19 @@ | ||
| { | ||
| "extends": "../../tsconfig.packages.build.json", | ||
| "compilerOptions": { | ||
| "baseUrl": "./", | ||
| "outDir": "./dist", | ||
| "rootDir": "./src" | ||
| }, | ||
| "references": [ | ||
| { "path": "../base-controller/tsconfig.build.json" }, | ||
| { "path": "../geolocation-controller/tsconfig.build.json" }, | ||
| { "path": "../messenger/tsconfig.build.json" } | ||
| { | ||
| "path": "../base-controller/tsconfig.build.json" | ||
| }, | ||
| { | ||
| "path": "../geolocation-controller/tsconfig.build.json" | ||
| }, | ||
| { | ||
| "path": "../messenger/tsconfig.build.json" | ||
| } | ||
| ], | ||
| "include": ["../../types", "./src"] | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,12 +1,15 @@ | ||
| { | ||
| "extends": "../../tsconfig.packages.json", | ||
| "compilerOptions": { | ||
| "baseUrl": "./" | ||
| }, | ||
| "references": [ | ||
| { "path": "../base-controller" }, | ||
| { "path": "../geolocation-controller" }, | ||
| { "path": "../messenger" } | ||
| { | ||
| "path": "../base-controller" | ||
| }, | ||
| { | ||
| "path": "../geolocation-controller" | ||
| }, | ||
| { | ||
| "path": "../messenger" | ||
| } | ||
| ], | ||
| "include": ["../../types", "./src"] | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I couldn't get
tscandtsc6working without this patch. With this patch, thetscandtsc6executables work as expected.