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

Transpile js #88

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Transpile js #88

wants to merge 4 commits into from

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.

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