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

[WIP] Add axum implementation #27

Closed
wants to merge 16 commits into from
Closed

[WIP] Add axum implementation #27

wants to merge 16 commits into from

Conversation

AndreyErmilov
Copy link
Contributor

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Sep 15, 2021

Codecov Report

Merging #27 (e2f63c6) into master (af352d3) will decrease coverage by 5.09%.
The diff coverage is 22.61%.

❗ Current head e2f63c6 differs from pull request most recent head d353d78. Consider uploading reports for the commit d353d78 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master      #27      +/-   ##
==========================================
- Coverage   71.00%   65.91%   -5.10%     
==========================================
  Files          41       44       +3     
  Lines         714      798      +84     
==========================================
+ Hits          507      526      +19     
- Misses        207      272      +65     
Impacted Files Coverage Δ
hitbox-axum/src/layer.rs 0.00% <0.00%> (ø)
hitbox-axum/src/service.rs 0.00% <0.00%> (ø)
hitbox-axum/src/cacheable.rs 52.77% <52.77%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update af352d3...d353d78. Read the comment docs.

@AndreyErmilov AndreyErmilov changed the title Add axum implementation [WIP] Add axum implementation Sep 15, 2021
@singulared
Copy link
Member

Closed in favour #25

@singulared singulared closed this Jul 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants