Skip to content

Add user option to use JPMS#2

Merged
lburaggi merged 5 commits intoPi4J:masterfrom
Adito5393:feature/runOnModulePath
Apr 25, 2021
Merged

Add user option to use JPMS#2
lburaggi merged 5 commits intoPi4J:masterfrom
Adito5393:feature/runOnModulePath

Conversation

@Adito5393
Copy link
Copy Markdown
Contributor

@Adito5393 Adito5393 commented Apr 13, 2021

Following up on issue #1. Here is a working concept of how the archetype would handle the module-info.

TODO:

  • write the archetype-post-generate.groovy file to delete the module-info.java if the user property useJavaPlatformModuleSystem=false - the pom.xml does not need to be changed;
  • update README.md with how the new parameters (useJavaPlatformModuleSystem, run.with.java.module and target.run.module.class) work;

The current PR is a working JPMS project, so feel free to test it and provide feedback!

@FDelporte FDelporte requested a review from lburaggi April 13, 2021 17:45
@Adito5393
Copy link
Copy Markdown
Contributor Author

The project is completed for testing with module-info and without (making use of useJavaPlatformModuleSystem project parameter)!

Will this PR be considered for merging? If not, I will not invest time in updating the docs.

@Adito5393 Adito5393 marked this pull request as ready for review April 22, 2021 09:31
@lburaggi lburaggi merged commit a836566 into Pi4J:master Apr 25, 2021
@lburaggi
Copy link
Copy Markdown
Collaborator

Good job

Copy link
Copy Markdown
Collaborator

@lburaggi lburaggi left a comment

Choose a reason for hiding this comment

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

Accepted

@FDelporte
Copy link
Copy Markdown
Member

@Adito5393 thanks for your contribution!!!

Regarding the documentation: this Maven Archetype is also described on https://pi4j.com/getting-started/developing-on-a-remote-pc/. Please, where applicable, update both the README in this repository and the website page on https://github.com/Pi4J/pi4j.github.io/blob/main/content/getting-started/developing-on-a-remote-pc.md.

@Adito5393 Adito5393 deleted the feature/runOnModulePath branch May 6, 2022 18:50
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.

3 participants