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

Rename Test Folders to Align with Module Names #661

Merged
merged 1 commit into from
Aug 24, 2022

Conversation

isVoid
Copy link
Contributor

@isVoid isVoid commented Aug 23, 2022

Description

#656 missed refactoring the test files to align with the names in the spatial module. Though it's not an explicit requirement in cuDF python, since the API surface of cuspatial is still manageable, we can try to enforce a similar folder structure in tests as the core module.

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@github-actions github-actions bot added the Python Related to Python code label Aug 23, 2022
@isVoid isVoid changed the title rename test folders to align with module names Rename Test Folders to Align with Module Names Aug 23, 2022
@isVoid isVoid added non-breaking Non-breaking change improvement Improvement / enhancement to an existing function labels Aug 23, 2022
@isVoid isVoid requested a review from thomcom August 23, 2022 21:19
@isVoid isVoid marked this pull request as ready for review August 23, 2022 21:19
@isVoid isVoid requested a review from a team as a code owner August 23, 2022 21:19
@isVoid
Copy link
Contributor Author

isVoid commented Aug 24, 2022

@gpucibot merge

@rapids-bot rapids-bot bot merged commit df3a163 into rapidsai:branch-22.10 Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvement / enhancement to an existing function non-breaking Non-breaking change Python Related to Python code
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants