microsoft / microsoft/hve-core

docs(security): Install-PSModules.ps1 help marks two live callers as (planned)

Open Beginner friendly
#2,916 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
1.5k
Forks
301
Avg merge
3d 3h
Merged PRs (30d)
92

Description

`scripts/security/Install-PSModules.ps1` marks two of its three callers as `(planned)` in the comment-based help, but both already call it.

Lines 49-50 of the `.NOTES` block read:

```
Called by: .devcontainer/scripts/on-create.sh (planned)
Called by: .github/workflows/copilot-setup-steps.yml (planned)
```

Both are live today. `.devcontainer/scripts/on-create.sh:87` runs `pwsh -NoProfile -File scripts/security/Install-PSModules.ps1 -Repository "${PSGALLERY_REPO}"`, and `.github/workflows/copilot-setup-steps.yml:92` runs `& "$env:GITHUB_WORKSPACE/scripts/security/Install-PSModules.ps1" -Scope CurrentUser -Repository $repo`.

The third entry, `.github/actions/setup-ps-modules/action.yml`, carries no tag and is correct.

Effect is small but pointed at contributors: anyone reading the help to work out where this script runs will conclude the devcontainer and the Copilot setup workflow do not use it yet, and that is no longer true.

Happy to send the two-line change dropping both `(planned)` tags if that is welcome.

Contributor guide

Open the contributing guide

Research direction

Open scripts/security/Install-PSModules.ps1 and inspect the .NOTES block around lines 49-50. Confirm the callers in .devcontainer/scripts/on-create.sh:87 and .github/workflows/copilot-setup-steps.yml:92, then remove the two incorrect (planned) tags while leaving the action.yml entry unchanged.

Written by the indexing model from the issue text.

Assessment

Tech stack
powershell
Domain
documentation
Issue type
Documentation
Difficulty
1/5
Estimated time
Under an hour
Activity status
Active
Clarity
Clearly specified
Newbie friendliness
94/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.