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.

Sign up for GitHub

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

Jump to bottom

[BUG] Open report doesn't use maven wrapper #541

Open
fbricon opened this issue Sep 30, 2021 · 1 comment
Open

[BUG] Open report doesn't use maven wrapper #541

fbricon opened this issue Sep 30, 2021 · 1 comment
Labels
Analyzing enhancement

Comments

@fbricon
Copy link
Contributor

fbricon commented Sep 30, 2021

Describe the bug
I tested the extension on a machine that doesn't have Maven installed globally. I have a project that has the mvnw wrapper instead. When clicking on the report icon in the status bar, I got an ugly error

To Reproduce
Steps to reproduce the behavior:

  1. don't have Maven installed globally
  2. open maven project using a mvnw wrapper
  3. click on the report icon
  4. See error

Expected behavior
Opening the report should favor using the wrapper if it exists, before using the global mvn command

Screenshots

Capture d’écran (1)

VSCode(please complete the following information):

  • OS: Win 10
  • VSCode version 1.60.2
  • Dependency Analytics Version 0.3.3
@Andrei-Paul
Copy link

It would be nice if it would honor "maven.executable.preferMavenWrapper": true

@rubensa rubensa mentioned this issue Apr 25, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Analyzing enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@fbricon @Andrei-Paul @IlonaShishov