Skip to content

Conversation

@sarsko
Copy link
Contributor

@sarsko sarsko commented Nov 22, 2025

Adds Shuttle support for the parking_lot library


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@@ -0,0 +1,20 @@
[package]
name = "shuttle-parking_lot"
version = "0.0.1"
Copy link
Member

Choose a reason for hiding this comment

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

This should be "0.12.5" right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Debatable, I think no.

In my head releasing version mirrors is independent from the code. Thus all wrappers will be pushed as 0.0.1, because this is a releasable non-colliding version for all the packages, meaning we can release it and depend on it without there being any confusion on what version its wrapping. We can also yank a version 0.0.1 that was released (accidentally or not) without any loss. It is thus "safe" to have version 0.0.1. Version "0.12.5" however has a version collision, and publishing it is a final action. It is not "safe" to have version 0.12.5.

Regardless of what we release, I think the crates should live in the repo as 0.0.1s

@@ -0,0 +1,22 @@
[package]
name = "shuttle-rand"
version = "0.0.1"
Copy link
Member

Choose a reason for hiding this comment

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

Should be 0.8 no?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Same as the comment on the parking lot version

@sarsko
Copy link
Contributor Author

sarsko commented Nov 26, 2025

The original plan was to use #233 to resolve the issue with poisoned locks. Upon further investigation this does not work, and we need to expose a lock which doesn't poison from Shuttle. Put up an issue here #237

@sarsko
Copy link
Contributor Author

sarsko commented Jan 12, 2026

All comments should be addressed

@sarsko sarsko force-pushed the parking_lot branch 3 times, most recently from 56ee338 to 6782e63 Compare January 12, 2026 22:01
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