-
Notifications
You must be signed in to change notification settings - Fork 192
Clean up project structure and bump version to 2.0 #397
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
Conversation
d69e963 to
436f404
Compare
Also clean up dependencies and make them consistent, and unify spacing across pom.xml files.
436f404 to
2711952
Compare
jmluy
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor comment/question but otherwise LGTM
| <version>0.0.1-SNAPSHOT</version> | ||
| <name>json-server-implementation </name> | ||
| <description>Example Spring Application of a json server </description> | ||
| <artifactId>ocpp16-json-server-example</artifactId> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not a bug but do you want to rename the directory for the server example to match the client?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not really. I renamed the OCPP-J directory, because it was sticking out like a sore thumb, but otherwise left existing directory names untouched.
The inconsistencies in the directory names here are minor, so I think renaming them is not worth the possible confusion.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, it's just a minor inconsistency.
Clean up the project structure, separating OCPP 1.6J and OCPP 1.6S support.
Rename the artifacts to more descriptive names.
Unify names and descriptions across all gradle and maven build files.
Update README.md with a description of the new project structure.
Bump the library version to 2.0 and the artifacts versions to 2.0.0.