Nieuwe Pull aanvraag

#730 Go async

Open

igalic wil 46 commits van go/async samenvoegen met master

update rocket and its dependencies to their async branch.
update our code to use rocket’s async APIs

👍 1

Codecov Report

Merging #730 into master will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #730      +/-   ##
==========================================
+ Coverage   39.08%   39.11%   +0.03%     
==========================================
  Files          73       73              
  Lines        9736     9643      -93     
  Branches     2227     2183      -44     
==========================================
- Hits         3805     3772      -33     
+ Misses       4879     4819      -60     
  Partials     1052     1052

igalic (Gemigreerd van github.com) beoordeeld 1 jaar geleden

src/routes/mod.rs

Plume_migration_agent commented 1 jaar geleden

this std::fs::read could (should?) be async.

elegaanz (Gemigreerd van github.com) beoordeeld 1 jaar geleden

src/routes/mod.rs

Plume_migration_agent commented 1 jaar geleden

It could with something like async-std, yes. But maybe add a todo and leave it for a future PR?

igalic refereerde aan deze pull request 7 maanden geleden

[Refactoring] Move DbConn out PlumeRocket #797

This pull request has changes conflicting with the target branch.

Cargo.lock

Cargo.toml

plume-models/Cargo.toml

src/main.rs

src/routes/posts.rs

src/routes/user.rs

Log in om deel te nemen aan deze discussie.

Geen beoordelaars

Geen label

Geen mijlpaal

Niet toegewezen

1 deelnemers

Notificaties

Vervaldatum

Geen vervaldatum ingesteld.

Afhankelijkheden

Deze pull-aanvraag heeft momenteel geen afhankelijkheden.