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.

Sign up for GitHub

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

Jump to bottom

PyO3: migrate to Bound for src/rust/engine/src/intrinsics/docker.rs #21484

Merged
merged 1 commit into from
Oct 3, 2024
Merged

PyO3: migrate to Bound for src/rust/engine/src/intrinsics/docker.rs #21484

merged 1 commit into from
Oct 3, 2024

Conversation

tdyas
Copy link
Contributor

@tdyas tdyas commented Oct 3, 2024

Migrate to pyo3::Bound smart pointer for src/rust/engine/src/intrinsics/docker.rs.

This PR adds an additional converation for Value from &Bound<'py, PyAny>.

@tdyas tdyas added category:internal CI, fixes for not-yet-released features, etc. release-notes:not-required PR doesn't require mention in release notes labels Oct 3, 2024
@tdyas tdyas requested review from benjyw and huonw October 3, 2024 03:18
@tdyas tdyas enabled auto-merge (squash) October 3, 2024 03:46
@tdyas tdyas merged commit a9f2bf4 into pantsbuild:main Oct 3, 2024
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@benjyw benjyw benjyw approved these changes

@huonw huonw Awaiting requested review from huonw

Assignees
No one assigned
Labels
category:internal CI, fixes for not-yet-released features, etc. release-notes:not-required PR doesn't require mention in release notes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tdyas @benjyw