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

Require 3.1.0+ version of Maven for Maven plugin #259

Merged
merged 1 commit into from
Jul 2, 2018

Conversation

lutovich
Copy link
Contributor

@lutovich lutovich commented Jul 2, 2018

Such runtime requirement exists because plugin uses APIs only available starting from 3.1.0 version of Maven. They are used for artifact resolution and come from the org.eclipse.aether package which does not exist in < 3.1.0 Maven versions.

Requirement is declared in a <prerequisites> section of the POM. This section is deprecated for regular projects and should only be used by plugins to declare a minimum Maven version. Old versions will make the plugin execution fail with an error:

[ERROR] Failed to execute goal c.d.s:spotless-maven-plugin:X.Y.Z:apply (default-cli) on project test: The plugin c.d.s:spotless-maven-plugin:X.Y.Z requires Maven version 3.1.0 -> [Help 1]

Also added a note in the README about the minimum required Maven version.

Resolves #257

@lutovich lutovich requested a review from nedtwigg July 2, 2018 17:23
Such runtime requirement exists because plugin uses APIs only
available starting from 3.1.0 version of Maven. They are used for
artifact resolution and come from the `org.eclipse.aether` package
which does not exist in < 3.1.0 Maven versions.

Requirement is declared in a `<prerequisites>` section of the POM.
This section is deprecated for regular projects and should only be
used by plugins to declare a minimum Maven version. Old versions will
make the plugin execution fail with an error:

```
[ERROR] Failed to execute goal c.d.s:spotless-maven-plugin:X.Y.Z:apply (default-cli) on project test: The plugin c.d.s:spotless-maven-plugin:X.Y.Z requires Maven version 3.1.0 -> [Help 1]
```

Also added a note in the README about the minimum
required Maven version.
@lutovich lutovich force-pushed the mvn-plugin-required-mvn-version branch from 1b39213 to 7818509 Compare July 2, 2018 17:25
@nedtwigg nedtwigg merged commit 5c862ce into diffplug:master Jul 2, 2018
@nedtwigg
Copy link
Member

nedtwigg commented Jul 2, 2018

Thanks @lutovich!

@lutovich lutovich deleted the mvn-plugin-required-mvn-version branch July 2, 2018 18:27
@nedtwigg nedtwigg mentioned this pull request Apr 9, 2019
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants