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

Replace local copyright check with pre-commit-hooks verify-copyright #48

Merged
merged 3 commits into from
Mar 18, 2024

Conversation

KyleFromNVIDIA
Copy link
Contributor

The local copyright.py script is bug-prone. Replace it with a more robust centralized script from pre-commit-hooks.

Issue: rapidsai/build-planning#30

The local copyright.py script is bug-prone. Replace it with a more
robust centralized script from pre-commit-hooks.

Issue: rapidsai/build-planning#30
@KyleFromNVIDIA KyleFromNVIDIA added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Mar 11, 2024
@KyleFromNVIDIA KyleFromNVIDIA requested a review from a team as a code owner March 11, 2024 21:35
@github-actions github-actions bot added the ci label Mar 11, 2024
Copy link
Member

@benfred benfred left a comment

Choose a reason for hiding this comment

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

looks good - thanks!

.pre-commit-config.yaml Outdated Show resolved Hide resolved
@benfred
Copy link
Member

benfred commented Mar 12, 2024

/merge

@rapids-bot rapids-bot bot merged commit 964f686 into rapidsai:branch-24.04 Mar 18, 2024
8 checks passed
divyegala pushed a commit to divyegala/cuvs that referenced this pull request Mar 20, 2024
…apidsai#48)

The local `copyright.py` script is bug-prone. Replace it with a more robust centralized script from `pre-commit-hooks`.

Issue: rapidsai/build-planning#30

Authors:
  - Kyle Edwards (https://github.com/KyleFromNVIDIA)

Approvers:
  - Ben Frederickson (https://github.com/benfred)
  - Ray Douglass (https://github.com/raydouglass)

URL: rapidsai#48
benfred pushed a commit to benfred/cuvs that referenced this pull request Apr 5, 2024
…apidsai#48)

The local `copyright.py` script is bug-prone. Replace it with a more robust centralized script from `pre-commit-hooks`.

Issue: rapidsai/build-planning#30

Authors:
  - Kyle Edwards (https://github.com/KyleFromNVIDIA)

Approvers:
  - Ben Frederickson (https://github.com/benfred)
  - Ray Douglass (https://github.com/raydouglass)

URL: rapidsai#48
difyrrwrzd added a commit to difyrrwrzd/cuvs that referenced this pull request Aug 10, 2024
…(#48)

The local `copyright.py` script is bug-prone. Replace it with a more robust centralized script from `pre-commit-hooks`.

Issue: rapidsai/build-planning#30

Authors:
  - Kyle Edwards (https://github.com/KyleFromNVIDIA)

Approvers:
  - Ben Frederickson (https://github.com/benfred)
  - Ray Douglass (https://github.com/raydouglass)

URL: rapidsai/cuvs#48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci improvement Improves an existing functionality non-breaking Introduces a non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants