Use maven wrapper if available#2934
Use maven wrapper if available#2934fenuks wants to merge 2 commits intodense-analysis:masterdense-analysis/ale:masterfrom fenuks:maven_helperCopy head branch name to clipboard
Conversation
|
Hello, since there are no change requests, can this get merged? :) |
|
This pull request has been automatically marked as stale because it has not been updated recently. Make sure to write tests and document your changes. See |
|
@w0rp Could you take look at this? I think it's good to go, changes had been reviewed, there are tests, and no conflicts (I've rebased it to sit on top of the latest upstream version, but since this pull request is closed, update is not shown). |
|
@fenuks can you reopen this PR? I cannot for some reason. I will merge this if you manage to reopen it. I had tested this and works fine. |
|
Sadly, I cannot. I guess the fastest way will be opening a new PR. |
This merge request adds
maven.vimhelper that extracts code for working with maven projects, adds detection formvnw(andmvnw.cmdon Windows) wrapper. If wrapper is available, it is used to determine root of the maven project, and if it is executable, it is also used instead of globalmvncommand to generate classpath list.Tests are included, based on analogous tests for
gradle.vimhelper.