Skip to content

docs(completions): document zsh fpath requirement for tab-completion (DX-5783)#41

Open
johnpmitsch wants to merge 1 commit into
mainfrom
dx-completions-zsh-docs
Open

docs(completions): document zsh fpath requirement for tab-completion (DX-5783)#41
johnpmitsch wants to merge 1 commit into
mainfrom
dx-completions-zsh-docs

Conversation

@johnpmitsch

Copy link
Copy Markdown
Collaborator

Summary

Brew installs the completion files, but zsh only autoloads a completion when its directory is on $fpath before compinit runs — so qn <TAB> still fell back to filename completion for many zsh users. This documents that requirement; no behavior change to the command.

  • README: both completions sections now state the zsh $fpath/compinit requirement and link the zsh manual; per-shell examples no longer assume an install method.
  • qn completions --help: self-documenting long_about (install-path + zsh nuances) plus per-shell install examples.

Closes DX-5783

Test plan

  • cargo test, cargo clippy --all-targets -- -D warnings, cargo fmt --check all pass
  • qn completions --help shows the zsh note + per-shell examples
  • zsh: write _qn into a dir, add it to $fpath before compinit, open a fresh shell → qn <TAB> completes subcommands

@johnpmitsch johnpmitsch force-pushed the dx-completions-zsh-docs branch 2 times, most recently from 2fcb020 to f7024ad Compare June 17, 2026 15:45
yorsant
yorsant previously approved these changes Jun 17, 2026
…(DX-5783)

Brew installs the completion files, but zsh only autoloads a completion
when its directory is on $fpath before compinit runs — so `qn <TAB>`
still fell back to filename completion for many zsh users.

Document the requirement in the README (both completions sections, linking
the zsh manual) and make `qn completions --help` self-documenting with a
long_about explaining install-path and zsh nuances plus per-shell install
examples. No behavior change to the command itself.
@johnpmitsch johnpmitsch force-pushed the dx-completions-zsh-docs branch from 5870d4f to 0426bc8 Compare June 17, 2026 15:52
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.

2 participants