Skip to content

[FAIL] feat: introdue order picker#490

Draft
MrCroxx wants to merge 1 commit intomainfrom
xx/order
Draft

[FAIL] feat: introdue order picker#490
MrCroxx wants to merge 1 commit intomainfrom
xx/order

Conversation

@MrCroxx
Copy link
Member

@MrCroxx MrCroxx commented May 15, 2024

What's changed and what's your intention?

Please explain IN DETAIL what the changes are in this PR and why they are needed. :D

Checklist

  • I have written the necessary rustdoc comments
  • I have added the necessary unit tests and integration tests
  • I have passed make all (or make fast instead if the old tests are not modified) in my local environment.

Related issues or PRs (optional)

Signed-off-by: MrCroxx <mrcroxx@outlook.com>
@MrCroxx MrCroxx added the feature New feature or request label May 15, 2024
@MrCroxx MrCroxx self-assigned this May 15, 2024
@codecov
Copy link

codecov bot commented May 15, 2024

Codecov Report

Attention: Patch coverage is 93.44262% with 20 lines in your changes are missing coverage. Please review.

Project coverage is 82.01%. Comparing base (0e3e55d) to head (b76927a).

Files Patch % Lines
foyer-memory/src/order.rs 93.97% 15 Missing ⚠️
foyer-bench/src/main.rs 61.53% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #490      +/-   ##
==========================================
+ Coverage   81.44%   82.01%   +0.57%     
==========================================
  Files          60       62       +2     
  Lines        7824     8120     +296     
==========================================
+ Hits         6372     6660     +288     
- Misses       1452     1460       +8     

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

@MrCroxx MrCroxx force-pushed the main branch 2 times, most recently from 3f7f53f to c009129 Compare September 12, 2025 05:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant