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

fix: adjusted copyright headers #304

Merged
merged 1 commit into from
May 3, 2024
Merged

fix: adjusted copyright headers #304

merged 1 commit into from
May 3, 2024

Conversation

matbmoser
Copy link
Contributor

@matbmoser matbmoser commented May 3, 2024

Why we create this PR?

As discussed and agreed in the latest PI planning BASF requested to be removed from the copyright for the further development.

As agreed added CGI in the copyright instead.

What we want to achieve with this PR?

Fix the copyright headers that were incorrect.

What is new?

Fixed

  • Copyright headers aligned with discussions


VOLUME data log

RUN mkdir -p ./log

Check notice

Code scanning / KICS

Multiple RUN, ADD, COPY, Instructions Listed Note

There are RUN instructions that could be grouped
@thorstendikmann
Copy link

As discussed and agreed - approved.

@matbmoser matbmoser added bug Something isn't working and removed bug Something isn't working labels May 3, 2024
@matbmoser matbmoser requested a review from saudkhan116 May 3, 2024 16:18
@thorstendikmann
Copy link

Revised and approved.

@saudkhan116
Copy link
Contributor

The files have only license header changed.
All good and approved!

@matbmoser matbmoser merged commit 3fbaa62 into main May 3, 2024
9 of 10 checks passed
@saudkhan116 saudkhan116 deleted the hotfix/remove-basf branch May 6, 2024 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants