diff options
author | bors[bot] <26634292+bors[bot]@users.noreply.github.com> | 2020-02-03 17:11:36 +0000 |
---|---|---|
committer | GitHub <[email protected]> | 2020-02-03 17:11:36 +0000 |
commit | b090ee5a65f9630146c2842bc51fcfcc8da08da1 (patch) | |
tree | 433ab3412e57a51ca170937f007aa36affc03ed6 | |
parent | 8d18b151e42aafaec713832f25f48aefcc0958c5 (diff) | |
parent | f0323de7e8a38113a4ceeba068b5c06b5b541182 (diff) |
Merge #3003
3003: Remove rollup-typescript r=matklad a=matklad
It seems like just calling typescript directly is simpler and more reliable?
@Veetaha what do you think about this approach?
Co-authored-by: Aleksey Kladov <[email protected]>
-rw-r--r-- | editors/code/package-lock.json | 10 | ||||
-rw-r--r-- | editors/code/package.json | 5 | ||||
-rw-r--r-- | editors/code/rollup.config.js | 4 |
3 files changed, 3 insertions, 16 deletions
diff --git a/editors/code/package-lock.json b/editors/code/package-lock.json index 261e36a02..353af06bf 100644 --- a/editors/code/package-lock.json +++ b/editors/code/package-lock.json | |||
@@ -61,16 +61,6 @@ | |||
61 | } | 61 | } |
62 | } | 62 | } |
63 | }, | 63 | }, |
64 | "@rollup/plugin-typescript": { | ||
65 | "version": "2.1.0", | ||
66 | "resolved": "https://registry.npmjs.org/@rollup/plugin-typescript/-/plugin-typescript-2.1.0.tgz", | ||
67 | "integrity": "sha512-7lXKGY06aofrceVez/YnN2axttFdHSqlUBpCJ6ebzDfxwLDKMgSV5lD4ykBcdgE7aK3egxuLkD/HKyRB5L8Log==", | ||
68 | "dev": true, | ||
69 | "requires": { | ||
70 | "@rollup/pluginutils": "^3.0.0", | ||
71 | "resolve": "^1.13.1" | ||
72 | } | ||
73 | }, | ||
74 | "@rollup/pluginutils": { | 64 | "@rollup/pluginutils": { |
75 | "version": "3.0.8", | 65 | "version": "3.0.8", |
76 | "resolved": "https://registry.npmjs.org/@rollup/pluginutils/-/pluginutils-3.0.8.tgz", | 66 | "resolved": "https://registry.npmjs.org/@rollup/pluginutils/-/pluginutils-3.0.8.tgz", |
diff --git a/editors/code/package.json b/editors/code/package.json index e4eae7b31..c0d8f0183 100644 --- a/editors/code/package.json +++ b/editors/code/package.json | |||
@@ -18,9 +18,9 @@ | |||
18 | "vscode": "^1.41.0" | 18 | "vscode": "^1.41.0" |
19 | }, | 19 | }, |
20 | "scripts": { | 20 | "scripts": { |
21 | "vscode:prepublish": "rollup -c", | 21 | "vscode:prepublish": "tsc && rollup -c", |
22 | "package": "vsce package", | 22 | "package": "vsce package", |
23 | "watch": "tsc -watch -p ./", | 23 | "watch": "tsc --watch", |
24 | "fmt": "tsfmt -r && tslint -c tslint.json 'src/**/*.ts' --fix" | 24 | "fmt": "tsfmt -r && tslint -c tslint.json 'src/**/*.ts' --fix" |
25 | }, | 25 | }, |
26 | "dependencies": { | 26 | "dependencies": { |
@@ -30,7 +30,6 @@ | |||
30 | "devDependencies": { | 30 | "devDependencies": { |
31 | "@rollup/plugin-commonjs": "^11.0.2", | 31 | "@rollup/plugin-commonjs": "^11.0.2", |
32 | "@rollup/plugin-node-resolve": "^7.1.1", | 32 | "@rollup/plugin-node-resolve": "^7.1.1", |
33 | "@rollup/plugin-typescript": "^2.0.0", | ||
34 | "@types/node": "^12.12.25", | 33 | "@types/node": "^12.12.25", |
35 | "@types/vscode": "^1.41.0", | 34 | "@types/vscode": "^1.41.0", |
36 | "rollup": "^1.31.0", | 35 | "rollup": "^1.31.0", |
diff --git a/editors/code/rollup.config.js b/editors/code/rollup.config.js index de6a3b2b7..f8d320f46 100644 --- a/editors/code/rollup.config.js +++ b/editors/code/rollup.config.js | |||
@@ -1,12 +1,10 @@ | |||
1 | import typescript from '@rollup/plugin-typescript'; | ||
2 | import resolve from '@rollup/plugin-node-resolve'; | 1 | import resolve from '@rollup/plugin-node-resolve'; |
3 | import commonjs from '@rollup/plugin-commonjs'; | 2 | import commonjs from '@rollup/plugin-commonjs'; |
4 | import nodeBuiltins from 'builtin-modules'; | 3 | import nodeBuiltins from 'builtin-modules'; |
5 | 4 | ||
6 | export default { | 5 | export default { |
7 | input: 'src/main.ts', | 6 | input: 'out/main.js', |
8 | plugins: [ | 7 | plugins: [ |
9 | typescript(), | ||
10 | resolve({ | 8 | resolve({ |
11 | preferBuiltins: true | 9 | preferBuiltins: true |
12 | }), | 10 | }), |