docs: stop sending Fish users the Bash wrapper function #3

Merged
jercik merged 1 commit from docs/fish-wrapper into main 2026-10-22 17:44:44 +00:00
Owner

Follow-up to #2: the --open wrapper function is bash/zsh syntax, so the README now scopes it to those shells (adding the ; bash needs before }) and gives fish its own function.

🤖 Generated with Claude Code

Follow-up to #2: the `--open` wrapper function is bash/zsh syntax, so the README now scopes it to those shells (adding the `;` bash needs before `}`) and gives fish its own function. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
docs: stop sending Fish users the Bash wrapper function
All checks were successful
Checks / pr-flow-canary (pull_request) Has been skipped
Checks / reuse (pull_request) Successful in 3s
commit-msg / commitlint (pull_request) Successful in 9s
Checks / quality-checks (26.5.0) (pull_request) Successful in 33s
Checks / quality-checks (24.15.0) (pull_request) Successful in 33s
Review / Review (pull_request_target) Successful in 2m7s
09d830685a
The README told fish users to paste a bash/zsh brace function into
config.fish, where it is a parse error. Scope that snippet to bash and
zsh, add a fish function, and add the `;` bash needs before the closing
brace.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Review 01M5JAVCG480WSKF8D7PXZ4XG6 — head 09d830685a9e3d1957d8fbc6d63cfd7fd87b7f68

Review — j4k-oss/worktree-add @ 94ff757b29

Scope: diff against base tree f0cf3699298b
Status: dispatched — coverage complete (5/5 slots terminal)
Facts: current review-wide projection

Computed under:

{
  "abandonment": "abandonment-v1",
  "anchor_recipe": 1,
  "batch_policy": "batch-v1",
  "coverage": "coverage-v3",
  "dispatch_policy": "dispatch-v2",
  "grounder_version": 1,
  "grounding_read_rule": "grounding-read-v1",
  "promotion_policy": "promotion-v1",
  "report": "report-v4",
  "tally": "tally-v1",
  "triage_settle": "triage-settle-v2"
}

Findings (0)

No findings survived.

Reviewed:

  • restated-sets (whole/default): no-claims
  • general-bug (whole/default): no-claims
  • writing-quality (whole/default): no-claims
  • project-docs (whole/default): no-claims
  • unix-cli (whole/default): no-claims
  • test-trimming: skipped-by-dispatch — no tests changed; only README.md modified
  • use-effect: skipped-by-dispatch — documentation-only change with no React code
  • react: skipped-by-dispatch — documentation-only change with no React code

Other claims

  • grounding-pending (0)
  • ungrounded (0)
  • rejected (0)
  • duplicate-of (0)
  • unadjudicated (0)

Coverage

Coverage pass: 01M5JAVVWXRTPAA9ASN4W5EQT4
Accounting: complete
Slot health: healthy

lens part arm unit status runs loss
restated-sets whole default no-claims 1 no
general-bug whole default no-claims 1 no
writing-quality whole default no-claims 1 no
project-docs whole default no-claims 1 no
unix-cli whole default no-claims 1 no
  • test-trimming — skipped-by-dispatch: no tests changed; only README.md modified
  • use-effect — skipped-by-dispatch: documentation-only change with no React code
  • react — skipped-by-dispatch: documentation-only change with no React code
<!-- review:summary --> **Review** `01M5JAVCG480WSKF8D7PXZ4XG6` — head `09d830685a9e3d1957d8fbc6d63cfd7fd87b7f68` # Review — j4k-oss/worktree-add @ 94ff757b291b Scope: diff against base tree `f0cf3699298b` Status: dispatched — coverage complete (5/5 slots terminal) Facts: current review-wide projection Computed under: ```json { "abandonment": "abandonment-v1", "anchor_recipe": 1, "batch_policy": "batch-v1", "coverage": "coverage-v3", "dispatch_policy": "dispatch-v2", "grounder_version": 1, "grounding_read_rule": "grounding-read-v1", "promotion_policy": "promotion-v1", "report": "report-v4", "tally": "tally-v1", "triage_settle": "triage-settle-v2" } ``` ## Findings (0) No findings survived. Reviewed: - restated-sets (whole/default): no-claims - general-bug (whole/default): no-claims - writing-quality (whole/default): no-claims - project-docs (whole/default): no-claims - unix-cli (whole/default): no-claims - test-trimming: skipped-by-dispatch — no tests changed; only README.md modified - use-effect: skipped-by-dispatch — documentation-only change with no React code - react: skipped-by-dispatch — documentation-only change with no React code ## Other claims - grounding-pending (0) - ungrounded (0) - rejected (0) - duplicate-of (0) - unadjudicated (0) ## Coverage Coverage pass: 01M5JAVVWXRTPAA9ASN4W5EQT4 Accounting: complete Slot health: healthy | lens | part | arm | unit status | runs | loss | | --- | --- | --- | --- | --- | --- | | restated-sets | whole | default | no-claims | 1 | no | | general-bug | whole | default | no-claims | 1 | no | | writing-quality | whole | default | no-claims | 1 | no | | project-docs | whole | default | no-claims | 1 | no | | unix-cli | whole | default | no-claims | 1 | no | - test-trimming — skipped-by-dispatch: no tests changed; only README.md modified - use-effect — skipped-by-dispatch: documentation-only change with no React code - react — skipped-by-dispatch: documentation-only change with no React code
jercik merged commit 8fd2b0eb33 into main 2026-10-22 17:44:44 +00:00
jercik deleted branch docs/fish-wrapper 2026-10-22 17:44:44 +00:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
j4k-oss/worktree-add!3
No description provided.