diff options
author | bors[bot] <26634292+bors[bot]@users.noreply.github.com> | 2020-02-15 22:14:42 +0000 |
---|---|---|
committer | GitHub <[email protected]> | 2020-02-15 22:14:42 +0000 |
commit | e4054f7e35a9a6c6e518066697567db92036607c (patch) | |
tree | 267844783734912d4a002a8144b653a8e66abb4c | |
parent | 58f4dcf79ecf3b8dcec83d46a27a29340900a0ef (diff) | |
parent | cc43f07e11e2a7715ab6eb14a18f23fc93e5fcdb (diff) |
Merge #3158
3158: Disable rollup warning r=matklad a=edwin0cheng
In https://rollupjs.org/guide/en/#output-exports
```
As with regular entry points, files that mix default and named exports will produce warnings. You can avoid the warnings by forcing all files to use named export mode via output.exports: "named".
```
This PR try added `output.exports: "named"` and disabe this warning. It is because vscode consume our plugins only by named functions such that it should be saved to disable it.
cc @Veetaha
Co-authored-by: Edwin Cheng <[email protected]>
-rw-r--r-- | editors/code/rollup.config.js | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/editors/code/rollup.config.js b/editors/code/rollup.config.js index f8d320f46..337385a24 100644 --- a/editors/code/rollup.config.js +++ b/editors/code/rollup.config.js | |||
@@ -18,6 +18,7 @@ export default { | |||
18 | external: [...nodeBuiltins, 'vscode'], | 18 | external: [...nodeBuiltins, 'vscode'], |
19 | output: { | 19 | output: { |
20 | file: './out/main.js', | 20 | file: './out/main.js', |
21 | format: 'cjs' | 21 | format: 'cjs', |
22 | exports: 'named' | ||
22 | } | 23 | } |
23 | }; | 24 | }; |