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

Drop extraneous dependencies from cudf conda recipe. #13406

Merged
merged 1 commit into from
May 22, 2023

Conversation

bdice
Copy link
Contributor

@bdice bdice commented May 22, 2023

Description

This PR drops two dependencies from the cudf conda recipe.

  • numba is only a run dependency and should not be listed in host.
  • fastavro is only a test dependency. It is listed in dependencies.yaml but does not need to be listed in meta.yaml as a run dependency. This is already listed as a testing dependency in pyproject.toml.

Checklist

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

@bdice bdice requested a review from a team as a code owner May 22, 2023 16:46
@github-actions github-actions bot added the conda label May 22, 2023
@bdice bdice added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels May 22, 2023
@bdice bdice mentioned this pull request May 22, 2023
@bdice
Copy link
Contributor Author

bdice commented May 22, 2023

/merge

@rapids-bot rapids-bot bot merged commit b57d1d6 into rapidsai:branch-23.06 May 22, 2023
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants