shell-pool / shell-pool/shpool

FR: shpool should have a flag to disable threading exit status back

Open
#153 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Rust
Stars
2k
Forks
67
Avg merge
3d 1h
Merged PRs (30d)
15

Description

Sometimes the exit status of the shell is what is interesting, and sometimes a better question to answer with the exit status is "was the attach successful". This latter question can use useful if you want to automatically run shpool attach when you shell into a remote machine and display an error if it is not successful. Threading the error status back down causes problems for these sorts of setups.

I'm not sure what to call the flag. Maybe --error-status-mode attach, where the options are attach and shell (with shell as the default to maintain current behavior).

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by tracing the shpool attach command and how the shell's exit status is currently propagated. Compare the proposed attach and shell modes, then identify the CLI parsing, status-handling, documentation, and test locations involved. Done means a flag selects attach-success status without changing the default shell-status behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
cli
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.