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

Cargo publish: properly convert path dependencies #328

Merged
merged 1 commit into from
Feb 27, 2025

Conversation

Tpt
Copy link
Contributor

@Tpt Tpt commented Feb 25, 2025

Use only the already published package instead of a hybrid mix of the local version and the published package

Issue #293

@Tpt Tpt force-pushed the tpt/cargo-publish-path branch 8 times, most recently from 61b186f to 042d88f Compare February 25, 2025 19:48
@Tpt Tpt force-pushed the tpt/cargo-publish-path branch 6 times, most recently from ad7ec30 to 2b3f2f9 Compare February 26, 2025 11:24
Use only the already published package instead of a hybrid mix of the local version and the published package

Issue #293
@Tpt Tpt force-pushed the tpt/cargo-publish-path branch from 2b3f2f9 to 93a2539 Compare February 26, 2025 12:32
@Tpt Tpt marked this pull request as ready for review February 26, 2025 12:54
@Tpt Tpt requested review from vsiles and srstevenson February 26, 2025 12:54
Copy link
Contributor

@vsiles vsiles left a comment

Choose a reason for hiding this comment

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

Lots of test for a single line fix. I love it ;)

@Tpt Tpt merged commit 14b812a into develop Feb 27, 2025
15 checks passed
@Tpt Tpt deleted the tpt/cargo-publish-path branch February 27, 2025 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants