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

remove .gitattributes #740

Merged
merged 1 commit into from
Jun 24, 2024

Conversation

jameslamb
Copy link
Member

Contributes to rapidsai/build-planning#31

Removes .gitattributes files.

These have been here since the beginning of the project (048d53e), for use with versioneer. Per the git docs (link), setting the attribute export-subst on a file via a .gitattributes tell git to replace placeholders in the file with some git information.

This is no longer done in _version.py files in this project, and this project no longer uses versioneer (#615). rapids-build-backend handles storing git commit information in the published packages.

Notes for Reviewers

For more details, see rapidsai/build-planning#31 (comment)

@jameslamb jameslamb added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Jun 18, 2024
@jameslamb jameslamb requested review from msarahan and bdice June 18, 2024 21:07
@jameslamb jameslamb requested a review from a team as a code owner June 18, 2024 21:07
@jameslamb
Copy link
Member Author

jameslamb commented Jun 24, 2024

/merge

(edit: I missed that this needs cucim-python-codeowners approval)

Copy link
Contributor

@grlee77 grlee77 left a comment

Choose a reason for hiding this comment

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

Thanks, @jameslamb! Looks like we missed removing this when we moved away from versioneer last year.

@jameslamb
Copy link
Member Author

/merge

@rapids-bot rapids-bot bot merged commit f25b2c6 into rapidsai:branch-24.08 Jun 24, 2024
45 checks passed
@jameslamb jameslamb deleted the remove-gitattributes branch June 24, 2024 16:25
@jakirkham jakirkham added this to the v24.08.00 milestone Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

4 participants