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

Feat: Add vendor_generic_rust_package helper #555

Merged
merged 1 commit into from
Feb 25, 2025

Conversation

tiran
Copy link
Collaborator

@tiran tiran commented Feb 25, 2025

The vendor_generic_rust_package function is a generic version of vendor_rust function. The existing function deals with Python packaging. The generic function works with any Rust package.

The new function is needed for wandb, which builds a Rust program, not a Python extension.

The `vendor_generic_rust_package` function is a generic version of
`vendor_rust` function. The existing function deals with Python
packaging. The generic function works with any Rust package.

The new function is needed for `wandb`, which builds a Rust program, not
a Python extension.

Signed-off-by: Christian Heimes <[email protected]>
@tiran tiran requested review from dhellmann and rd4398 February 25, 2025 11:59
Copy link
Contributor

@rd4398 rd4398 left a comment

Choose a reason for hiding this comment

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

Overall looks good! I have added a comment which is more of a clarification question.

Copy link
Contributor

@rd4398 rd4398 left a comment

Choose a reason for hiding this comment

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

LGTM

@mergify mergify bot merged commit e1b03dc into python-wheel-build:main Feb 25, 2025
80 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.

2 participants