Skip to content

Conversation

@Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Oct 4, 2024

This (partially) removes the need for us to use a blocking reqwest client, which will be refactored in a follow-up PR.

@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 38 lines in your changes missing coverage. Please review.

Project coverage is 89.03%. Comparing base (828f52e) to head (9888fd1).
Report is 8 commits behind head on main.

Files with missing lines Patch % Lines
src/bin/monitor.rs 0.00% 28 Missing ⚠️
src/db.rs 0.00% 10 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #9578      +/-   ##
==========================================
- Coverage   89.10%   89.03%   -0.07%     
==========================================
  Files         289      289              
  Lines       29057    29076      +19     
==========================================
- Hits        25890    25888       -2     
- Misses       3167     3188      +21     

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

@Turbo87 Turbo87 merged commit 5faabd6 into rust-lang:main Oct 4, 2024
10 checks passed
@Turbo87 Turbo87 deleted the async-monitor branch October 4, 2024 13:34
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