Skip to content
This repository was archived by the owner on Jan 4, 2023. It is now read-only.

Transpile js#88

Open
braunse wants to merge 4 commits intoimba:masterfrom
braunse:transpile-js
Open

Transpile js#88
braunse wants to merge 4 commits intoimba:masterfrom
braunse:transpile-js

Conversation

@braunse
Copy link

@braunse braunse commented Apr 15, 2021

The compiled javascript uses the ?? and ?. operators which are not
compatible with the Node version that VS Code bundles. To avoid this,
the compiled javascript is postprocessed with Babel to replace these
operators by expressions that are supported by Node v12.18.3.

Fixes: #86

Note

Since the build process does not work on my machine, this PR is
dependent on #87, so it includes the commits from that PR at this point.
I created a separate PR since the changes from #87 should be considered
independently of the problem that this PR is meant to address.

braunse added 4 commits April 14, 2021 22:53
The `vscode.d.ts` file is not installed by the `postinstall` script of
the `vscode` package anymore. It is now contained in `@types/vscode` on
NPM.

See https://code.visualstudio.com/updates/v1_36#_splitting-vscode-package-into-typesvscode-and-vscodetest
The `imba build` command referenced in the `build` script does not
work. What actually needs to be built seem to be the
`client/dist/src/index.js` and `server/dist/src/index.js` files.
This change builds these files explicitly.
The compiled javascript uses the `??` and `?.` operators which are not
compatible with the Node version that VS Code bundles. To avoid this,
the compiled javascript is postprocessed with Babel to replace these
operators by expressions that are supported by Node v12.18.3.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

language server crashes immediately on launch

1 participant

Comments