Skip to content

fix(cli): tell PowerShell users to dot-source the installed completion script - #3272

Merged
SchoolyB merged 1 commit into
grayscale-lang:patch-v0.13.0from
SAY-5:fix/powershell-completion-hint
Oct 25, 2026
Merged

SchoolyB merged 1 commit into
grayscale-lang:patch-v0.13.0from
SAY-5:fix/powershell-completion-hint

Conversation

@SAY-5

@SAY-5 SAY-5 commented Oct 25, 2026

Copy link
Copy Markdown
Contributor

Related issue

Closes #3081

Summary

  • PowerShell has no completion directory it loads on its own, so after writing gray.ps1 the install line now says to add . "<path>" to the profile instead of reporting it as installed. The other shells keep the existing message.
  • I could not run this on Windows; the change is in the line installCompletionScript prints, covered by a unit test on the new helper.

Type of change

  • Bug fix
  • New feature
  • Refactor
  • Tests
  • Documentation
  • CI/Build

Breaking changes

None.

Checklist

  • make build compiles with zero warnings
  • Tests added/updated (unit, integration, or both as appropriate)
  • No @ before module names in any text (it tags GitHub users)
  • Added yourself to the Contributors: section of the header of each source file you changed

…n script

Signed-off-by: Sai Asish Y <say.apm35@gmail.com>
@SchoolyB SchoolyB added the PR: In Review This PR is being reviewed label Oct 25, 2026
@SchoolyB
SchoolyB changed the base branch from main to patch-v0.13.0 October 25, 2026 17:10

@SchoolyB SchoolyB left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀 Thanks for another contribution @SAY-5

@SchoolyB SchoolyB added PR: Ready This PR is ready pull request is ready to merge and removed PR: In Review This PR is being reviewed labels Oct 25, 2026
@SchoolyB
SchoolyB merged commit a9749d4 into grayscale-lang:patch-v0.13.0 Oct 25, 2026
7 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: Ready This PR is ready pull request is ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: PowerShell completion install reports success but the script is never loaded

2 participants