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

ref: Update actix integration to actix-web 4 #437

Merged
merged 8 commits into from
Mar 2, 2022
Merged

Conversation

Swatinem
Copy link
Member

@Swatinem Swatinem commented Feb 28, 2022

This also includes the following drive-by changes:

  • Make sure our examples are more consistent with usage of release_name/debug options.
  • Simplify transaction naming / avoid intermediate String allocation.
  • Update the Changelog.

supercedes #316
fixes #388
fixes #327
fixes #308

@Swatinem Swatinem requested a review from a team February 28, 2022 12:48
@codecov-commenter
Copy link

Codecov Report

Merging #437 (2810263) into master (871f727) will decrease coverage by 0.01%.
The diff coverage is 75.00%.

❗ Current head 2810263 differs from pull request most recent head 28bdf6d. Consider uploading reports for the commit 28bdf6d to get more accurate results

@@            Coverage Diff             @@
##           master     #437      +/-   ##
==========================================
- Coverage   80.52%   80.51%   -0.02%     
==========================================
  Files          73       73              
  Lines        8293     8302       +9     
==========================================
+ Hits         6678     6684       +6     
- Misses       1615     1618       +3     

@Swatinem Swatinem merged commit 761e27c into master Mar 2, 2022
@Swatinem Swatinem deleted the ref/update-actix branch March 2, 2022 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants