Skip to content

Conversation

@MrCroxx
Copy link
Member

@MrCroxx MrCroxx commented Jul 8, 2025

Signed-off-by: MrCroxx [email protected]## 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

Tried #1059 with OpenDAL, do some tiny refactorings.

Checklist

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

Related issues or PRs (optional)

#1060
#1059
apache/opendal#6366

@MrCroxx MrCroxx added this to the v0.18 milestone Jul 8, 2025
@MrCroxx MrCroxx self-assigned this Jul 8, 2025
@codecov
Copy link

codecov bot commented Jul 8, 2025

Codecov Report

Attention: Patch coverage is 42.85714% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
foyer/src/hybrid/error.rs 42.85% 4 Missing ⚠️
Files with missing lines Coverage Δ
foyer/src/hybrid/error.rs 58.33% <42.85%> (-4.53%) ⬇️

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@MrCroxx MrCroxx merged commit 4b50f6d into main Jul 8, 2025
44 checks passed
@MrCroxx MrCroxx deleted the xx/error branch July 8, 2025 06:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants