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 added this to Inbox in v5.0.2 via automation May 22, 2021
@XhmikosR XhmikosR added this to Inbox in v4.6.1 via automation May 22, 2021
@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 moved this from Inbox to Review in v5.0.2 Jun 4, 2021
v5.0.2 automation moved this from Review to Approved Jun 4, 2021
@XhmikosR XhmikosR merged commit 232a40a into main Jun 8, 2021
v5.0.2 automation moved this from Approved to Done Jun 8, 2021
@XhmikosR XhmikosR deleted the main-xmr-vnu branch June 8, 2021 05:44
@XhmikosR XhmikosR moved this from Inbox to Needs manual backport in v4.6.1 Jun 17, 2021
@XhmikosR XhmikosR moved this from Needs manual backport to Cherry-picked/Manually backported in v4.6.1 Jun 22, 2021
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
@XhmikosR XhmikosR moved this from Cherry-picked/Manually backported to Shipped in v4.6.1 Jul 21, 2021
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
No open projects
v4.6.1
Shipped
v5.0.2
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants