Skip to content

Commit

Permalink
docs: cranelift
Browse files Browse the repository at this point in the history
  • Loading branch information
alexandervantrijffel committed Oct 27, 2024
1 parent a6f9c2c commit 40c7c36
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
5 changes: 3 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,9 @@ This repository offers some Rust snippets that can be useful when studying the l

Dependencies:

- Install the command runner [just](https://github.com/casey/just?tab=readme-ov-file#installation).
- Install the test runner [cargo-nexttest](https://nexte.st/).
- Install the command runner [just](https://github.com/casey/just?tab=readme-ov-file#installation)
- Install the nextest runner [cargo-nexttest](https://nexte.st/)
- For improved incrememtal build times, install the nightly toolchain to support the cranelift backend: `rustup toolchain install nightly`

Run the snippets in debug mode:

Expand Down
2 changes: 1 addition & 1 deletion src/async_structured_concurrency.rs
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ use std::sync::RwLock;
/// With this approach, we don't need types that are Send like Arc to share the database and
/// service dependencies, and no async move is needed. This greatly improves the developer UX.
/// The test_demo test is configured to use the Tokio current thread runtime to emulate thread-per-core.

Check warning on line 13 in src/async_structured_concurrency.rs

View workflow job for this annotation

GitHub Actions / validate

item in documentation is missing backticks
/// The total duration of the test is 1.53s on my machine which proves that the 1000 requests are
/// The total duration of the test is 1.08s on my machine which proves that the requests are
/// processed concurrently. The spawned tasks within the inner moro scope are executed within the same
/// thread, these tasks are not moved between threads and therefore the called future doees not need
/// to be Send.
Expand Down

0 comments on commit 40c7c36

Please sign in to comment.