FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Added user-friendly installation instructions by admercs · Pull Request #2141 · PowerShell/PSScriptAnalyzer · GitHub

/ PSScriptAnalyzer Public

Added user-friendly installation instructions - #2141

Merged
Andy Jordan (andyleejordan) merged 3 commits into
PowerShell:mainfrom
admercs:readme_simple_install
Mar 19, 2026
ConversationCommits (3)ChecksFiles changed
Merged

Added user-friendly installation instructions#2141
Andy Jordan (andyleejordan) merged 3 commits into
PowerShell:mainfrom
admercs:readme_simple_install

Conversation

Adam Erickson (admercs) commented Nov 13, 2025

Copy link
Copy Markdown
Contributor

PR Summary

PR Checklist

Copilot AI review requested due to automatic review settings November 13, 2025 18:27
Copilot AI reviewed Nov 13, 2025
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Pull Request Overview

This PR adds inline PowerShell commands to the README.md file to provide users with a quick installation method for PSScriptAnalyzer directly from the documentation, complementing the existing link to the detailed installation guide.

  • Added inline PowerShell code snippet for installing PSScriptAnalyzer from PowerShell Gallery
  • Provided command to set PSGallery as a trusted repository before installation

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md Outdated
Comment thread README.md

```powershell
Set-PSRepository -Name PSGallery -InstallationPolicy Trusted
Install-Module -Name PSScriptAnalyzer

Copilot AI Nov 13, 2025

Copy link

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

The -Scope parameter should be specified for Install-Module to clarify where the module will be installed. Without it, PowerShell will attempt to install to AllUsers scope which requires elevated permissions, potentially causing the command to fail for non-admin users.

Consider updating to:

Install-Module -Name PSScriptAnalyzer -Scope CurrentUser

This allows users to install the module without requiring administrator privileges.

Suggested change
Install-Module -Name PSScriptAnalyzer
Install-Module -Name PSScriptAnalyzer -Scope CurrentUser

Copilot uses AI. Check for mistakes.

Sean Wheeler (sdwheeler) Mar 18, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Andy Jordan (@andyleejordan) We should use PSResourceGet commands.

Andy Jordan (andyleejordan) Mar 18, 2026

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Hopefully the linked instructions can be updated too.

Apply suggestion from @Copilot
4d0ce91 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Andy Jordan (andyleejordan) commented Mar 18, 2026

Copy link
Copy Markdown
Member

Sean Wheeler (@sdwheeler) is this reasonable or should be defaulting to Install-PSResource?

Andy Jordan (andyleejordan) requested changes Mar 18, 2026
View reviewed changes

Andy Jordan (andyleejordan) 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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Update it to Install-PSResource please!

Christoph Bergmeister (bergmeister) commented Mar 18, 2026

Copy link
Copy Markdown
Collaborator

Sean Wheeler (@sdwheeler) is this reasonable or should be defaulting to Install-PSResource?

I think having Install-Module and Install-PSResource is best as not everyone is on latest PowerShellGet, especially Windows PowerShell users

Andy Jordan (andyleejordan) approved these changes Mar 19, 2026
View reviewed changes

Andy Jordan (andyleejordan) 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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I'll defer to Christoph Bergmeister (@bergmeister) and agree since "easy" instructions means "just works"

Andy Jordan (andyleejordan) merged commit 45e3e6c into PowerShell:main Mar 19, 2026
4 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
Andy Jordan (andyleejordan) andyleejordan approved these changes
Christoph Bergmeister (bergmeister) Awaiting requested review from bergmeister bergmeister is a code owner
Sean Wheeler (sdwheeler) Awaiting requested review from sdwheeler

Assignees

No one assigned

Labels

Area - Documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

Back | FazBrowse Home | New Git URL