Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

9/26/2023 PM Publish #10465

Merged
merged 6 commits into from
Sep 26, 2023
Merged

9/26/2023 PM Publish #10465

merged 6 commits into from
Sep 26, 2023

Conversation

Taojunshen
Copy link
Contributor

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.

sdwheeler and others added 6 commits September 25, 2023 17:01
@sdwheeler @michaeltlombardi
Fixes #10440 - Explain how to use search scopes (#10457)
48f7fbc 
* Explain how to use search scopes

* Apply suggestions from review

---------

Co-authored-by: Mikey Lombardi (He/Him) <[email protected]>
@sdwheeler
Fixes #10297 - Update install docs for supported methods (#10460)
4cccd39 
* Update Linux install docs

* Update install docs for supported methods
@michaeltlombardi @sdwheeler
(GH-10459) Reorganize grouping expression docs (#10461)
e9f7059 
* (GH-10459) Reorganize grouping expression docs

Prior to this change, the documentation for using grouped expressions
in _about_Operators_ included two subheadings:

1. Grouping assignment statements
1. Piping grouped expressions

However, it also defined a third use-case for grouping, which showed
how to use the grouping operator to use the output of a command in
an expression. This use case was documented before the subheadings and
after a sentence indicating there's ways to use the grouping operator
beyond overriding operator precedence.

Further, the section for grouping assignment statements was very long
compared to the other use-cases and placed in the middle, despite being
less commonly used than the others.

This change:

- Adds a subheading for the first use case to make it equivalent to the
  others in the information hierarchy.
- Moves the section for piping grouped expressions to before the section
  for grouping assignment statements.
- Resolves #10459
- Fixes AB#163845

* Update reference/5.1/Microsoft.PowerShell.Core/About/about_Operators.md

---------

Co-authored-by: Sean Wheeler <[email protected]>
@sdwheeler
Format color values so they don't get localized (#10462)
0180d78
@mikefrobbins
Updated azps quick filter link (#10463)
8462394 
Co-authored-by: Mike F. Robbins <[email protected]>
@sdwheeler
Fix typos (#10464)
a10798f
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit a10798f:

✅ Validation status: passed

File Status Preview URL Details
reference/5.1/Microsoft.PowerShell.Core/About/about_Operators.md ✅Succeeded View (powershell-5.1)
reference/5.1/Microsoft.PowerShell.Utility/Write-Host.md ✅Succeeded View (powershell-5.1)
reference/7.2/Microsoft.PowerShell.Core/About/about_ANSI_Terminals.md ✅Succeeded View (powershell-7.2)
reference/7.2/Microsoft.PowerShell.Core/About/about_Operators.md ✅Succeeded View (powershell-7.2)
reference/7.2/Microsoft.PowerShell.Utility/Write-Host.md ✅Succeeded View (powershell-7.2)
reference/7.3/Microsoft.PowerShell.Core/About/about_ANSI_Terminals.md ✅Succeeded View (powershell-7.3)
reference/7.3/Microsoft.PowerShell.Core/About/about_Operators.md ✅Succeeded View (powershell-7.3)
reference/7.3/Microsoft.PowerShell.Utility/Write-Host.md ✅Succeeded View (powershell-7.3)
reference/7.4/Microsoft.PowerShell.Core/About/about_ANSI_Terminals.md ✅Succeeded View (powershell-7.4)
reference/7.4/Microsoft.PowerShell.Core/About/about_Operators.md ✅Succeeded View (powershell-7.4)
reference/7.4/Microsoft.PowerShell.Utility/Write-Host.md ✅Succeeded View (powershell-7.4)
reference/docs-conceptual/how-to-use-docs.md ✅Succeeded View (>=powershell-5.1)
reference/docs-conceptual/install/install-alpine.md ✅Succeeded View (>=powershell-5.1)
reference/docs-conceptual/install/install-debian.md ✅Succeeded View (>=powershell-5.1)
reference/docs-conceptual/install/Installing-PowerShell-on-macOS.md ✅Succeeded View (>=powershell-5.1)
reference/docs-conceptual/install/Installing-PowerShell-on-Windows.md ✅Succeeded View (>=powershell-5.1)
reference/docs-conceptual/install/install-rhel.md ✅Succeeded View (>=powershell-5.1)
reference/docs-conceptual/install/install-ubuntu.md ✅Succeeded View (>=powershell-5.1)
reference/docs-conceptual/media/how-to-use-docs/search-scope.gif ✅Succeeded View (>=powershell-5.1)
reference/includes/ubuntu-support.md ✅Succeeded View (>=powershell-5.1)
reference/module/index.md ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@Taojunshen Taojunshen merged commit c2b8f3a into live Sep 26, 2023
hassan20990
hassan20990 reviewed Sep 27, 2023
View reviewed changes
Copy link

@hassan20990 hassan20990 left a comment

Choose a reason for hiding this comment

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

good

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hassan20990 hassan20990 hassan20990 left review comments

@sdwheeler sdwheeler Awaiting requested review from sdwheeler sdwheeler is a code owner

@michaeltlombardi michaeltlombardi Awaiting requested review from michaeltlombardi michaeltlombardi is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Taojunshen @hassan20990 @sdwheeler @michaeltlombardi @mikefrobbins