Skip to content

Remove references to old Maven versions. - #194

Merged
elharo merged 2 commits into
masterfrom
maven2
Jun 20, 2023
Merged

elharo merged 2 commits into
masterfrom
maven2

Conversation

@elharo

@elharo elharo commented Jun 19, 2023

Copy link
Copy Markdown
Contributor

In 2023 assume everyone is on 3.3.1 or later.

@elharo
elharo requested a review from michael-o June 19, 2023 20:27
@slachiewicz

Copy link
Copy Markdown
Member

I hope You remember discussion on ML about minimum supported Maven version in all our plugins - so far 3.2.5 is minimal for users https://maven.apache.org/docs/history.html#maven-3-6-x-and-before and You strongly voted against setting it to 3.5.4

@elharo

elharo commented Jun 19, 2023

Copy link
Copy Markdown
Contributor Author

What we support and what we spend time calling out in documentation are two different things. Maven 2 is definitely dead.

@slachiewicz

Copy link
Copy Markdown
Member

Remove references to old Maven versions. In 2023 assume everyone is on 3.3.1 or later. if You say that this is true - so we can bump minimal supported Maven version for plugins to 3.3.1 and not be locked to 3.2.5 ;-)

@michael-o michael-o left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While I agree with @slachiewicz, I think we can do this and not waste more time for another PR in a year. People won't say thank you.

@elharo
elharo merged commit ae3a423 into master Jun 20, 2023
@elharo
elharo deleted the maven2 branch June 20, 2023 10:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants