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

chore: Updated legal distribution information #151

Merged
merged 5 commits into from
May 27, 2024

Conversation

adkumar1
Copy link
Contributor

@adkumar1 adkumar1 commented May 24, 2024

Pre-review checks

Please ensure to do as many of the following checks as possible, before asking for committer review:

@almadigabor
Copy link
Contributor

I don't think this is the right method to do it. Other teams don't copy the files explicitly in the Dockerfile but include them in the pom.xml to be part of the built .jar file. Could you try adjusting your pom.xml like here and see if you build the jar locally, does it contain the 3 files then?

@adkumar1
Copy link
Contributor Author

@almadigabor : We have fixed it.

@almadigabor almadigabor merged commit 253e942 into eclipse-tractusx:main May 27, 2024
5 checks passed
@almadigabor almadigabor deleted the legal_info branch May 27, 2024 11:02
@adkumar1 adkumar1 mentioned this pull request May 27, 2024
37 tasks
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.

fix: USERNAME variable does not exist in Dockerfile TRG 7.05 - Legal information for distributions
3 participants