Skip to content

Comments

Fix Find Errors#641

Merged
jabbate19 merged 18 commits intomainfrom
find_fixes
Nov 10, 2025
Merged

Fix Find Errors#641
jabbate19 merged 18 commits intomainfrom
find_fixes

Conversation

@jabbate19
Copy link
Collaborator

What type of PR is this?

/kind bug

What this PR does / why we need it:

Fix Bug

Which issue(s) this PR fixes:

Fixes #428

@codecov
Copy link

codecov bot commented Feb 21, 2024

Codecov Report

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

Project coverage is 73.05%. Comparing base (533e178) to head (ca72e14).
Report is 32 commits behind head on main.

Files Patch % Lines
implants/lib/eldritch/src/file/find_impl.rs 98.65% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #641      +/-   ##
==========================================
+ Coverage   72.15%   73.05%   +0.89%     
==========================================
  Files         174      175       +1     
  Lines       11937    12342     +405     
==========================================
+ Hits         8613     9016     +403     
- Misses       3110     3111       +1     
- Partials      214      215       +1     

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

@jabbate19 jabbate19 requested a review from hulto February 21, 2024 03:17
@jabbate19 jabbate19 requested a review from hulto February 22, 2024 19:30
@KCarretto
Copy link
Collaborator

Will leave this review to @hulto

use crate::runtime::Message;

#[tokio::test]
#[cfg(unix)]
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we add windows support here too?

@jabbate19 jabbate19 requested a review from hulto May 4, 2025 19:53
@jabbate19
Copy link
Collaborator Author

Failure appears unrelated

hulto
hulto previously approved these changes May 4, 2025
jabbate19 and others added 2 commits November 9, 2025 12:04
Message Enum usage changed since last time this branch was worked on. After merging main into this, the tests were breaking
@jabbate19 jabbate19 merged commit 9d4c08a into main Nov 10, 2025
4 checks passed
@jabbate19 jabbate19 deleted the find_fixes branch November 10, 2025 00:36
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.

file.find permissions errors and unexpected behavior.

3 participants