-
Notifications
You must be signed in to change notification settings - Fork 16
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
Feat: Setup new Maven Module for CleaningServiceDummy #449
Feat: Setup new Maven Module for CleaningServiceDummy #449
Conversation
Do we need a DOCKER_NOTICE.md like in the other modules? |
* SPDX-License-Identifier: Apache-2.0 | ||
******************************************************************************/ | ||
|
||
package com.catenax.bpdm.cleaning |
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.
I'm wondering should the package be something below org.eclipse.tractusx.bpdm as it's part of the OpenSource though just a sample? And regarding Maven it's part of the group "org.eclipse.tractusx".
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.
I changed the package name to org.eclipse.tractusx.bpdm.cleaning
Looks good! |
LGTM |
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.
Since this pull request introduces a new Docker notice file it should be referenced in the list of notice files in the README
I also think the commits can be merged into one
…p new Maven Module
b44a7a8
to
89b7ba3
Compare
I linked the Docker notice into the README file. |
Description
Create maven module for the Cleaning Service Dummy.
I used org.eclipse.tractusx.bpdm.cleaning as base package name because org.eclipse.tractusx.bpdm.cleaning-service-dummy seems a little bit long.
At the moment no persistence to a database is configured.
Solves
#421