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.

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

10/13/2023 PM Publish #10529

Merged
merged 8 commits into from
Oct 13, 2023
Merged

10/13/2023 PM Publish #10529

merged 8 commits into from
Oct 13, 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 8 commits October 12, 2023 18:40
Removes incorrect assertion that functions must use `begin`, `process`, and `end` if any named block is used. 

This implies these blocks are required even if they are empty. Assertion is contradicted later in the document when discussing `process`.
* Update Getting-WMI-Objects--Get-CimInstance-.md

added full command and modified values according to windows 11.

I edited this article 
https://learn.microsoft.com/en-us/powershell/scripting/samples/getting-wmi-objects--get-ciminstance-?view=powershell-7.3

* Apply suggestions from code review

---------

Co-authored-by: Sean Wheeler <[email protected]>
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 3d6f12b:

✅ Validation status: passed

File Status Preview URL Details
reference/5.1/Microsoft.PowerShell.Core/About/about_Functions_Advanced_Methods.md ✅Succeeded View (powershell-5.1)
reference/5.1/Microsoft.PowerShell.Core/About/about_WMI.md ✅Succeeded View (powershell-5.1)
reference/5.1/Microsoft.PowerShell.Core/About/about_WQL.md ✅Succeeded View (powershell-5.1)
reference/5.1/Microsoft.PowerShell.Management/Get-Content.md ✅Succeeded View (powershell-5.1)
reference/5.1/PSReadLine/About/about_PSReadLine_Functions.md ✅Succeeded View (powershell-5.1)
reference/7.2/CimCmdlets/Get-CimSession.md ✅Succeeded View (powershell-7.2)
reference/7.2/Microsoft.PowerShell.Core/About/about_Functions_Advanced_Methods.md ✅Succeeded View (powershell-7.2)
reference/7.2/Microsoft.PowerShell.Management/Get-Content.md ✅Succeeded View (powershell-7.2)
reference/7.2/Microsoft.PowerShell.Utility/Get-Variable.md ✅Succeeded View (powershell-7.2)
reference/7.2/PSReadLine/About/about_PSReadLine_Functions.md ✅Succeeded View (powershell-7.2)
reference/7.3/Microsoft.PowerShell.Core/About/about_CimSession.md ✅Succeeded View (powershell-7.3)
reference/7.3/Microsoft.PowerShell.Core/About/about_Functions_Advanced_Methods.md ✅Succeeded View (powershell-7.3)
reference/7.3/Microsoft.PowerShell.Management/Get-Content.md ✅Succeeded View (powershell-7.3)
reference/7.3/PSReadLine/About/about_PSReadLine_Functions.md ✅Succeeded View (powershell-7.3)
reference/7.4/Microsoft.PowerShell.Core/About/about_CimSession.md ✅Succeeded View (powershell-7.4)
reference/7.4/Microsoft.PowerShell.Core/About/about_Functions_Advanced_Methods.md ✅Succeeded View (powershell-7.4)
reference/7.4/Microsoft.PowerShell.Management/Get-Content.md ✅Succeeded View (powershell-7.4)
reference/7.4/Microsoft.PowerShell.Management/Set-ItemProperty.md ✅Succeeded View (powershell-7.4)
reference/7.4/PSReadLine/About/about_PSReadLine_Functions.md ✅Succeeded View (powershell-7.4)
reference/docfx.json ✅Succeeded
reference/docs-conceptual/developer/cmdlet/security-parameters.md ✅Succeeded View (>=powershell-5.1)
reference/docs-conceptual/samples/Getting-WMI-Objects--Get-CimInstance-.md ✅Succeeded View (>=powershell-5.1)

For more details, please refer to the build report.

For any questions, please:

@Taojunshen Taojunshen merged commit dc76d3c into live Oct 13, 2023
22 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.

4 participants