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

[green-code-initiative/ecoCode#336] Adds Maven Wrapper #67

Merged
merged 2 commits into from
Sep 6, 2024

Conversation

jycr
Copy link
Contributor

@jycr jycr commented Sep 1, 2024

This Maven Wrapper is an easy way to ensure a developer has everything necessary to build this project.

Executed command to add wrapper to project:

mvn wrapper:wrapper -Dmaven=3.9.9

This PR fix green-code-initiative/creedengo-rules-specifications#336 for ecoCode-java plugin

Also, migrate scripts to use Maven Wrapper (to ensure consistency of Maven version used in different environments)

@jycr jycr requested a review from dedece35 September 1, 2024 13:33
@jycr jycr self-assigned this Sep 1, 2024
Copy link
Member

@dedece35 dedece35 left a comment

Choose a reason for hiding this comment

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

ok for me.
could you please update the only missing thing : CHANGELOG.md ?

jycr added 2 commits September 2, 2024 16:14
This Maven Wrapper is an easy way to ensure a developer has everything necessary to build this project.

Executed command to add wrapper to project:

```sh
mvn wrapper:wrapper -Dmaven=3.9.9
```
… version used across different environments)
@jycr jycr force-pushed the feature/maven-wrapper branch from 5300df1 to b1abd7b Compare September 2, 2024 14:16
@jycr
Copy link
Contributor Author

jycr commented Sep 2, 2024

ok for me. could you please update the only missing thing : CHANGELOG.md ?

Done :)

@jycr jycr requested a review from dedece35 September 2, 2024 14:17
Copy link

sonarqubecloud bot commented Sep 2, 2024

Copy link
Member

@dedece35 dedece35 left a comment

Choose a reason for hiding this comment

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

ok for me ... I create a Kanban ticket to create the same PR on each other ecoCode project and upgrade check_requirements tool finally.
please see ticket https://github.com/orgs/green-code-initiative/projects/2/views/1?pane=issue&itemId=78854739

another idea : think if it is possible to upgrade this maven wrapper system to create an common and abstract way to make it easier to upgrade all projects

@jycr jycr merged commit 2329fd8 into main Sep 6, 2024
2 checks passed
@jycr jycr deleted the feature/maven-wrapper branch September 6, 2024 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants