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

fix: change Maven version from 3.5.1 to 3.8.6 as 3.5.1 doesn't exist:) #4140

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

PetrakovichVictoria
Copy link
Collaborator

The version 3.5.1 doesn't exist, see here: https://maven.apache.org/docs/history.html#maven-3-8-x

3.8.6 is chosen by advice from @CherepanovAleksei as we use this version.

Copy link

@CherepanovAleksei CherepanovAleksei left a comment

Choose a reason for hiding this comment

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

LGTM

@CherepanovAleksei CherepanovAleksei dismissed their stale review April 19, 2024 10:15

need to verify versions...

Copy link
Member

@koshachy koshachy left a comment

Choose a reason for hiding this comment

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

Hey @PetrakovichVictoria
Thanks for your contribution!

I'll wait @CherepanovAleksei for an approval.

@koshachy koshachy self-assigned this Apr 23, 2024
@CherepanovAleksei
Copy link

ftr, we need to revise the versions - the proposed version refers to the mvn version, but not to the version maven compiler plugin (https://mvnrepository.com/artifact/org.apache.maven.plugins/maven-compiler-plugin)

@koshachy koshachy marked this pull request as draft April 23, 2024 09:34
@PetrakovichVictoria
Copy link
Collaborator Author

We'll decide whether we should update this version or not. Anyway, it's not critical at all, so let it be a draft for a while.

JFYI: When configuring Kotlin in a Java project, the line with the version number is not added at all:) We're investigating if it's a bug.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants