-
Notifications
You must be signed in to change notification settings - Fork 267
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
Clarification in README - maintained versions, issues and PR creation #832
Conversation
According to clarification we can only test with the latest Maven on GitHub it will save time for executing
What do you think @slachiewicz, @stefanseifert, @ajarmoniuk, @sultan |
i would say let's move forward and be clear on the minimal maven version needed to run the plugin. |
I think we need to check the minimal version supported as well as a current version - which can be the latest release or some other one we choose. But without testing with the minimal version we can not be sure if we really support it. |
Maybe we should explicitly say that we recommend Maven 3.8.6 (plus link) and add note to verify if issue can be reproduced also with latest version of Maven |
Maven 4 API is not compatible backward to Maven 3, so it is for discussion later. |
I know that when we will testing only with latest versions of Maven some of issue can be detected. But only checking minimum version doesn't make us sure that will be work with all other older versions like 3.3.x, 3.5.x, 3.6.x When someone detect an issue with Maven 3.5.x - I want not spend time for resolving it. |
Exactly it is my idea and my proposition in change. |
Fair enough, I think you're right. |
According to clarification we can only test with the latest Maven on GitHub it will save time for executing