Skip to content
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

Improve vnu-jar.js #34075

Merged
merged 4 commits into from
Jun 8, 2021
Merged

Improve vnu-jar.js #34075

merged 4 commits into from
Jun 8, 2021

Conversation

XhmikosR
Copy link
Member

  • switch to execFile for the java version command
  • simplify our ignores and removed no longer needed ones
  • remove --no-langdetect since it doesn't seem to trigger any issues any more

* switch to `execFile` for the java version command
* simplify our ignores and removed no longer needed ones
* remove `--no-langdetect` since it doesn't seem to trigger any issues any more
@XhmikosR XhmikosR marked this pull request as ready for review June 4, 2021 04:10
@XhmikosR XhmikosR requested a review from a team as a code owner June 4, 2021 04:10
@XhmikosR XhmikosR merged commit 232a40a into main Jun 8, 2021
@XhmikosR XhmikosR deleted the main-xmr-vnu branch June 8, 2021 05:44
XhmikosR added a commit that referenced this pull request Jun 22, 2021
Improve and clean up vnu-jar.js
XhmikosR added a commit that referenced this pull request Jun 24, 2021
Improve and clean up vnu-jar.js
XhmikosR added a commit that referenced this pull request Jul 8, 2021
Improve and clean up vnu-jar.js
XhmikosR added a commit that referenced this pull request Jul 21, 2021
Improve and clean up vnu-jar.js
marvin-hinkley-vortx pushed a commit to Vortx-Inc/bootstrap that referenced this pull request Aug 18, 2021
* switch to `execFile` for the java version command
* simplify our ignores and removed no longer needed ones
* remove `--no-langdetect` since it doesn't seem to trigger any issues any more
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants