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

Apply updated license headers #118

Merged
merged 1 commit into from
Jan 2, 2024
Merged

Conversation

stes
Copy link
Member

@stes stes commented Jan 1, 2024

Please merge #117 first. This PR applies new license headers to all files, based on the config in #117 .

The commit is done via a bot account, so the cla check might fail as before. This is done to keep the contribution counts correct.

Copy link

cla-bot bot commented Jan 1, 2024

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: CEBRA.
This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email email@example.com
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

Copy link
Member Author

@stes stes left a comment

Choose a reason for hiding this comment

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

looked through the files, looks good. the pre-commit made a few minor corrections to three changed files (minor formatting or whitespace) which I would also keep included in this commit.

@stes
Copy link
Member Author

stes commented Jan 1, 2024

This notice,
image

is expected due to the bot account.

@stes stes force-pushed the stes/apply-license-headers branch from 17f43c7 to ee0a62c Compare January 1, 2024 19:19
Copy link

cla-bot bot commented Jan 1, 2024

Thank you for your contribution. We require contributors to sign our Contributor License Agreement (CLA). We do not have a signed CLA on file for you. In order for us to review and merge your code, please sign our CLA here. After you signed, you can comment on this PR with @cla-bot check to trigger another check.

@stes
Copy link
Member Author

stes commented Jan 1, 2024

@cla-bot check

@cla-bot cla-bot bot added the CLA signed label Jan 1, 2024
Copy link

cla-bot bot commented Jan 1, 2024

Thanks for tagging me. I looked for a signed form under your signature again, and updated the status on this PR. If the check was successful, no further action is needed. If the check was unsuccessful, please see the instructions in my first comment.

@stes stes force-pushed the stes/apply-license-headers branch from 7d5563c to d11aaf1 Compare January 2, 2024 22:46
@stes stes requested a review from MMathisLab January 2, 2024 22:47
@stes stes force-pushed the stes/apply-license-headers branch from d11aaf1 to 6f537d3 Compare January 2, 2024 23:19
@MMathisLab MMathisLab merged commit 02e43f7 into main Jan 2, 2024
11 checks passed
@MMathisLab MMathisLab deleted the stes/apply-license-headers branch January 2, 2024 23:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants