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 to rc1 or pinned revisions #9

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

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?

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