-
-
Notifications
You must be signed in to change notification settings - Fork 68
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
Simplify vscode-java-test installation instructions in README #596
Comments
Unfortunately, the |
The Mason registry also depends on this vsix asset, and @williamboman had the following to say:
See mason-org/mason-registry#3083 (comment). From that same pull-request thread, someone created a project to build and host See mason-org/mason-registry#3083 (comment). I'll wait on this issue, and update the |
A quick update on this. The Mason registry is sourcing these pre-built artifacts from the Open VSX registry:
I can work on a PR for this issue when I find time. |
Problem Statement
I recently setup this plugin for debugging JUnit tests with vscode-java-test, and ran into an issue following the instructions in the README. Particularly with the
npm run build-plugin
step sincevscode-java-test
depends on a SNAPSHOT version of JDTLS.See microsoft/vscode-java-test#1580 and microsoft/vscode-java-test#1581 for details.
I noticed looking at @mfussenegger's configuration, linked to from the Wiki, that they also had trouble with this step:
https://github.com/mfussenegger/dotfiles/blob/833d634251ebf3bf7e9899ed06ac710735d392da/vim/.config/nvim/ftplugin/java.lua#L109-L110
See discussion #329 for other problems with
vscode-java-test
bundles.Building the bundles for
vscode-java-test
may require a certain version of Java,npm
/ Node.js, and fail for an unknown reason.Ideas or possible solutions
A better way could be to download the pre-built bundles from the latest release.
For example, we can download the vscjava.vscode-java-test-0.39.0.vsix file, extract it, and specify the path to
/extension/server/
.SEE ALSO: Discussion #521
The text was updated successfully, but these errors were encountered: