-
Notifications
You must be signed in to change notification settings - Fork 2
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
Locked dependencies for publishing #9
Conversation
Signed-off-by: Darwin Boersma <[email protected]>
be8cf94
to
1c09c33
Compare
We’ve released rc2 with release in a week most likely, so maybe it’s worth waiting?
…On Fri, Nov 22, 2024, at 2:06 PM, Darwin Boersma wrote:
***@***.**** commented on this pull request.
In Cargo.toml <#9 (comment)>:
> @@ -8,17 +8,17 @@ version = "0.0.1"
edition = "2021"
[dependencies]
-any_spawner = { git = "https://github.com/leptos-rs/leptos", features = ["futures-executor"] }
-throw_error = { git = "https://github.com/leptos-rs/leptos" }
-hydration_context = { git = "https://github.com/leptos-rs/leptos" }
+any_spawner = { git = "https://github.com/leptos-rs/leptos", rev = "84bdd6b56892cc859b512b0de1dc139f83375f5d" , features = ["futures-executor"] }
Indeed, this does work in the leptos-spin template fermyon/leptos-spin#48 <fermyon/leptos-spin#48> -- I tested it by swapping the current pinned commit of leptos_wasi to my branch with `leptos_wasi = { git = "https://github.com/ogghead/leptos_wasi", branch = "rc1-locked-deps", optional = true }` and the Spin app spins up while referring to rc1 versions of Leptos crates
—
Reply to this email directly, view it on GitHub <#9 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABVBTCMN7ESSUABWIL6DZLD2B6THJAVCNFSM6AAAAABSJTAQYKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDINJVHAZDSMJSGU>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
I think at least using |
Checking into this and upgraded deps to rc2, but I am actually not seeing any release of https://crates.io/crates/any_spawner in the last 5 months. Leptos exports the Executor from |
Signed-off-by: Darwin Boersma <[email protected]>
Good catch! Well I guess we should wait for the definite release which will include your changes so, WDYT? |
Agreed! I can keep an eye out and update this PR with 0.7 dependencies + publish a 0.7 version of this crate after release if that makes sense? |
Now that we're versioning any_spawner through leptos, let's tweak that so that we can do a nonbreaking change after we release 0.7 and can still demo for the meetup Sunday! |
Signed-off-by: Darwin Boersma <[email protected]>
@benwis I updated this PR with changes to use leptos any_spawner functionality -- we should be able to simply update the versions to 0.7 with no other changes required after 0.7 releases with these modifications. Hopefully that is what you had in mind :) |
Perfect, thanks @ogghead |
Do you guys have a demo ready or you want me to present what I presented to wasmCloud? :D |
That sounds great to me @raskyld! |
We've released leptos_wasi v0.1.1. Nice work everyone! |
This satisfies the first step of #8 with rc1 locked versions
@raskyld @benwis I appreciate reviews when ya'll have time!