Skip to content

Isolated Processes 9: Repair ProcessReader API#188

Merged
marti4d merged 2 commits into
mainfrom
isolated_process/09-fix-process-reader
Jul 7, 2026
Merged

Isolated Processes 9: Repair ProcessReader API#188
marti4d merged 2 commits into
mainfrom
isolated_process/09-fix-process-reader

Conversation

@marti4d

@marti4d marti4d commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@marti4d

marti4d commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator Author

@afranchuk I also can't add you as a reviewer for whatever reason, but... Tada! 🎉

@marti4d
marti4d force-pushed the isolated_process/09-fix-process-reader branch 5 times, most recently from 9467308 to c65f61d Compare June 25, 2026 14:34
@marti4d
marti4d force-pushed the isolated_process/09-fix-process-reader branch from c65f61d to 92f580a Compare June 25, 2026 15:55
@Jake-Shadle
Jake-Shadle removed their request for review June 25, 2026 17:58

@afranchuk afranchuk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks great! Let's rename or broaden the process-reader crate.

Comment thread crates/process-reader/src/linux/error.rs
Comment thread crates/process-reader/src/linux/mod.rs
Comment thread crates/process-reader/src/linux/wrapper.rs
Comment thread crates/process-reader/Cargo.toml
Comment thread src/linux/process_inspection/mod.rs
@marti4d

marti4d commented Jul 6, 2026

Copy link
Copy Markdown
Collaborator Author

@gabrielesvelto This should be a pretty quick review, since Alex has already approved it :)

Comment thread crates/process-reader/src/lib.rs Outdated
@gabrielesvelto

Copy link
Copy Markdown
Contributor

Looking at this now. I assume it obsoletes PR #187 right?

@gabrielesvelto gabrielesvelto left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, thanks to both of you

@marti4d
marti4d force-pushed the isolated_process/09-fix-process-reader branch from f8b35ba to eee48b2 Compare July 7, 2026 13:40
@marti4d
marti4d merged commit 45f5459 into main Jul 7, 2026
16 checks passed
@marti4d
marti4d deleted the isolated_process/09-fix-process-reader branch July 7, 2026 13:45
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.

3 participants