Skip to content

Conversation

@Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Oct 4, 2024

... because we want to reduce our usage of spawn_blocking()

`ThreadRng` is not `Send` which is required to be able to use it across await-points with `tokio`.
@Turbo87 Turbo87 added C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear A-backend ⚙️ labels Oct 4, 2024
@codecov
Copy link

codecov bot commented Oct 4, 2024

Codecov Report

Attention: Patch coverage is 0% with 8 lines in your changes missing coverage. Please review.

Project coverage is 88.92%. Comparing base (cf322b4) to head (7512308).
Report is 6 commits behind head on main.

Files with missing lines Patch % Lines
src/admin/populate.rs 0.00% 8 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #9583   +/-   ##
=======================================
  Coverage   88.92%   88.92%           
=======================================
  Files         289      289           
  Lines       29118    29117    -1     
=======================================
  Hits        25892    25892           
+ Misses       3226     3225    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Turbo87 Turbo87 merged commit dadafc5 into rust-lang:main Oct 4, 2024
10 checks passed
@Turbo87 Turbo87 deleted the async-populate branch October 4, 2024 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-backend ⚙️ C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant