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 pre-commit copyright check #82

Merged
merged 3 commits into from
Jun 3, 2024
Merged

Conversation

bdice
Copy link
Contributor

@bdice bdice commented May 31, 2024

This PR is a small fix for an error I ran into locally on the copyright check.

@bdice bdice requested a review from a team as a code owner May 31, 2024 15:53
@jakirkham
Copy link
Member

Thanks Bradley! 🙏

Should we add copyright lines to other files like these as well?

# Generates the input files used by the pynvjitlink binding test suite

extern __device__ float undef(float a, float b);

@bdice
Copy link
Contributor Author

bdice commented May 31, 2024

@jakirkham The copyright checker will enforce that in the future, next time those files are modified. If you would like to add them in this PR, feel free to push.

@bdice
Copy link
Contributor Author

bdice commented Jun 3, 2024

I'm going to merge this as-is. We can gradually fix the files in this repo as they are edited, which is the same strategy we've been using across RAPIDS.

@bdice
Copy link
Contributor Author

bdice commented Jun 3, 2024

/merge apparently this repo doesn't use /merge comments

@bdice bdice merged commit 9c4c2e8 into rapidsai:main Jun 3, 2024
49 checks passed
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.

3 participants