docs(environments): describe the Linux sleep inhibitor as it now runs - #6373
Conversation
#6372 replaced the inhibitor's `sleep infinity` with `cat` on a pipe the guard holds, so the release is the pipe closing rather than a signal; the environment guide still showed the old command. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0134iUMxmGuXiG1LzPgfZVnv
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Claude finished @Hmbown's task in 42s —— View job Review: docs(environments) — Linux sleep inhibitor description
Summary This is a documentation-only fix (+5/-3, single file) correcting I cross-checked the updated text against
No correctness, provider/model-routing, reuse, test-coverage, or security concerns — this only touches prose in a markdown file. No code changes to review. I don't have anything to flag here; the update accurately reflects current behavior. |
Summary
Devin's analysis note on #6372:
docs/ENVIRONMENTS.mdstill showed the Linux inhibitor assystemd-inhibit … sleep infinity, while since #6372 the command iscatreading a pipe the guard holds and the release is that pipe closing. The guide now says so, and states that nothing is left behind.No-Issue: docs follow-up to #6372's review note; one line of the environment guide.
Testing
Docs only; no code or generated file changes. CI's light-change path covers the markdown checks.
cargo fmt --all -- --check(n/a, no Rust changes)cargo clippy --workspace --all-targets --all-features --locked(n/a)cargo test --workspace --all-features --locked(n/a)Checklist
🤖 Generated with Claude Code
https://claude.ai/code/session_0134iUMxmGuXiG1LzPgfZVnv
Generated by Claude Code