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

Locked dependencies for publishing #9

Merged
merged 3 commits into from
Nov 28, 2024
Merged

Conversation

ogghead
Copy link
Contributor

@ogghead ogghead commented Nov 22, 2024

This satisfies the first step of #8 with rc1 locked versions

@raskyld @benwis I appreciate reviews when ya'll have time!

Cargo.toml Outdated Show resolved Hide resolved
@ogghead ogghead requested a review from raskyld November 22, 2024 22:07
@benwis
Copy link
Contributor

benwis commented Nov 22, 2024 via email

@raskyld
Copy link
Collaborator

raskyld commented Nov 23, 2024

I think at least using rc2 makes sense to have everything clean and avoid git refs

@ogghead
Copy link
Contributor Author

ogghead commented Nov 23, 2024

I think at least using rc2 makes sense to have everything clean and avoid git refs

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 any_spawner but not the CustomExecutor used by this crate. It seems that crate might need to be released -- or Leptos might need to reexport that CustomExecutor trait -- before we can move off the git revision locked version of any_spawner

Signed-off-by: Darwin Boersma <[email protected]>
@raskyld
Copy link
Collaborator

raskyld commented Nov 24, 2024

Good catch! Well I guess we should wait for the definite release which will include your changes so, WDYT?

@ogghead
Copy link
Contributor Author

ogghead commented Nov 24, 2024

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?

@ogghead ogghead changed the title Locked dependencies to rc1 or pinned revisions Locked dependencies for publishing Nov 26, 2024
@benwis
Copy link
Contributor

benwis commented Nov 27, 2024

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!

@ogghead
Copy link
Contributor Author

ogghead commented Nov 28, 2024

@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 :)

@benwis
Copy link
Contributor

benwis commented Nov 28, 2024

Perfect, thanks @ogghead

@benwis benwis merged commit d7618b1 into leptos-rs:main Nov 28, 2024
1 check passed
@ogghead ogghead deleted the rc1-locked-deps branch November 28, 2024 01:28
@raskyld
Copy link
Collaborator

raskyld commented Nov 28, 2024

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!

Do you guys have a demo ready or you want me to present what I presented to wasmCloud? :D

@ogghead
Copy link
Contributor Author

ogghead commented Nov 28, 2024

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!

@benwis
Copy link
Contributor

benwis commented Nov 28, 2024

We've released leptos_wasi v0.1.1. Nice work everyone!

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.

3 participants