Skip to content

Conversation

@MrCroxx
Copy link
Member

@MrCroxx MrCroxx commented Aug 5, 2025

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

AS IS

For OpenDAL use case. FYI: apache/opendal#6366

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)

apache/opendal#6366

@MrCroxx MrCroxx added this to the v0.19 milestone Aug 5, 2025
@MrCroxx MrCroxx self-assigned this Aug 5, 2025
@MrCroxx MrCroxx added the feature New feature or request label Aug 5, 2025
@codecov
Copy link

codecov bot commented Aug 5, 2025

Codecov Report

❌ Patch coverage is 82.60870% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
foyer-storage/src/filter.rs 82.60% 4 Missing ⚠️
Files with missing lines Coverage Δ
foyer-storage/src/filter.rs 83.05% <82.60%> (-0.29%) ⬇️

... and 3 files 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 e9f2c57 into main Aug 5, 2025
44 checks passed
@MrCroxx MrCroxx deleted the xx/esf branch August 5, 2025 14:00
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: Done

Development

Successfully merging this pull request may close these issues.

2 participants