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

Update dev dependencies #153

Merged
merged 1 commit into from
Sep 23, 2024
Merged

Update dev dependencies #153

merged 1 commit into from
Sep 23, 2024

Conversation

jayvdb
Copy link
Contributor

@jayvdb jayvdb commented Sep 7, 2024

No description provided.

@jayvdb jayvdb changed the title Update dev dependencies Update dev dependency async-std Sep 17, 2024
@jayvdb jayvdb changed the title Update dev dependency async-std Update dev dependencies Sep 17, 2024
@@ -33,9 +33,9 @@ nanorand = { version = "0.7", features = ["getrandom"], optional = true }
#flume-test = { path = "../flume-test" }
crossbeam-channel = "0.5.5"
crossbeam-utils = "0.8.10"
criterion = "0.3.4"
criterion = { version = "0.5.1", default-features = false }
Copy link
Contributor Author

Choose a reason for hiding this comment

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

default features needs to be disabled to avoid

   Compiling criterion v0.5.1
error: Rayon cannot be used when targeting wasi32. Try disabling default features.
  --> /home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/criterion-0.5.1/src/lib.rs:31:1
   |
31 | compile_error!("Rayon cannot be used when targeting wasi32. Try disabling default features.");
   | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

error: could not compile `criterion` (lib) due to 1 previous error

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@zesterer , the workflows should be green now.

@zesterer
Copy link
Owner

Thanks!

@zesterer zesterer merged commit d487a62 into zesterer:master Sep 23, 2024
5 checks passed
@jayvdb jayvdb deleted the update-dev-deps branch September 23, 2024 10:05
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