Skip to content

Conversation

Taojunshen
Copy link
Contributor

@Taojunshen Taojunshen commented Nov 15, 2024

Prior to this change, the article for installing PowerShell on Windows used a single codeblock to show readers how to install PowerShell and PowerShell preview with WinGet by ID. On the webpage, this results in a single codeblock with a copy button. If a reader uses the copy button and pastes the copied code into their terminal, both commands will run. If a user wanted to install only one edition, they need to edit the copied code to remove the line they don't want.

This change:

PR Summary

PR Checklist

  • Descriptive Title: This PR's title is a synopsis of the changes it proposes.
  • Summary: This PR's summary describes the scope and intent of the change.
  • Contributor's Guide: I have read the contributors guide.
  • Style: This PR adheres to the style guide.

Prior to this change, the article for installing PowerShell on Windows used a
single codeblock to show readers how to install PowerShell and PowerShell preview
with WinGet by ID. On the webpage, this results in a single codeblock with a copy
button. If a reader uses the copy button and pastes the copied code into their
terminal, both commands will run. If a user wanted to install only one edition,
they need to edit the copied code to remove the line they don't want.

This change:

- Separates the codeblocks for installing PowerShell with WinGet and specifying
  the ID parameter for PowerShell and PowerShell preview, so that readers may
  copy either codeblock and paste into their terminal instead of copying both.
- Fixes #11534
- Fixes AB#343802
Copy link
Contributor

Learn Build status updates of commit fe9d48e:

✅ Validation status: passed

File Status Preview URL Details
reference/docs-conceptual/install/Installing-PowerShell-on-Windows.md ✅Succeeded View (>=powershell-5.1)

For more details, please refer to the build report.

For any questions, please:

@Taojunshen Taojunshen merged commit a4d2623 into live Nov 15, 2024
37 checks passed
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