By default, let Maven download Allure Command Line (fixes #141) #142
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Since this is a Maven plugin, we can reasonably expect that the user
already has configured maven correctly to reach Maven Central (or other
repositories) in some way. Therefore, instead of trying to deduce a way
to download our commandline tools from Maven settings, just ask Maven
kindly to do the download for us.
Context
Since Allure has been available from Maven Central for quite a while, it should be less of a hassle for most users to just let Maven deal with the downloading of the command line artifact. The old code is still left as-is, but only gets used if the
allureDownloadUrl
is specified at all.Additionally, this drops the embedded copy of allure-commandline, which makes this plugin quite a bit lighter.
WARNING: This breaks compatibility with Allure < 2.8.0 when not specifying
<allureDownloadUrl>https://dl.bintray.com/qameta/generic/io/qameta/allure/allure/%s/allure-%s.zip</allureDownloadUrl>
in the plugin configuration, but I consider this a minor problem, since that version is quite old nowadays.Checklist