Skip to content

fix: install PowerShell completions - #212

Open
LouisDeconinck wants to merge 1 commit into
getsotto:mainfrom
LouisDeconinck:fix/powershell-completions-139
Open

fix: install PowerShell completions#212
LouisDeconinck wants to merge 1 commit into
getsotto:mainfrom
LouisDeconinck:fix/powershell-completions-139

Conversation

@LouisDeconinck

Copy link
Copy Markdown
Contributor

Summary

Install the bundled PowerShell completion script when an existing PowerShell completions directory is available, while preserving the current hint otherwise.

Checks

  • git diff --check
  • Manual review for Windows PowerShell 5.1-compatible syntax

Closes #139

@github-actions

Copy link
Copy Markdown

Thanks for your first pull request to Sotto, and welcome.

Two things that trip up newcomers:

  • Every commit needs a Signed-off-by line under the Developer Certificate of Origin - PRs with unsigned commits cannot be merged. Sign off with git commit -s; if your branch already has unsigned commits, run git rebase --signoff main and push again.
  • Please run cargo fmt --all --check and cargo clippy --workspace --all-targets -- -D warnings before pushing, and add tests for new behaviour (cargo test --workspace runs the suite).

The full guide is in CONTRIBUTING.md - a maintainer will review shortly.

Signed-off-by: Louis Deconinck <louis.dck@gmail.com>
@LouisDeconinck
LouisDeconinck force-pushed the fix/powershell-completions-139 branch from 78ecd9a to 21db236 Compare September 11, 2026 11:04
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.

install.ps1: install PowerShell completions when a destination already exists

2 participants