Skip to content

Fixes #12638 - Fix broken examples#12639

Merged
sdwheeler merged 2 commits intoMicrosoftDocs:mainfrom
sdwheeler:sdw-w543607-i12638-winevent
Jan 6, 2026
Merged

Fixes #12638 - Fix broken examples#12639
sdwheeler merged 2 commits intoMicrosoftDocs:mainfrom
sdwheeler:sdw-w543607-i12638-winevent

Conversation

@sdwheeler
Copy link
Copy Markdown
Collaborator

@sdwheeler sdwheeler commented Jan 6, 2026

PR Summary

Fix broken examples

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 contributor's guide.
  • Style: This PR adheres to the style guide.

Copilot AI review requested due to automatic review settings January 6, 2026 16:36
@learn-build-service-prod
Copy link
Copy Markdown
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 409f1b6:

✅ Validation status: passed

File Status Preview URL Details
reference/5.1/Microsoft.PowerShell.Diagnostics/Get-WinEvent.md ✅Succeeded View (powershell-5.1)
reference/7.4/Microsoft.PowerShell.Diagnostics/Get-WinEvent.md ✅Succeeded View (powershell-7.4)
reference/7.5/Microsoft.PowerShell.Diagnostics/Get-WinEvent.md ✅Succeeded View (powershell-7.5)
reference/7.6/Microsoft.PowerShell.Diagnostics/Get-WinEvent.md ✅Succeeded View (powershell-7.6)

For more details, please refer to the build report.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR attempts to fix broken examples in the Get-WinEvent cmdlet documentation across multiple PowerShell versions (5.1, 7.4, 7.5, 7.6). The primary focus was on fixing HTML entity encoding issues in XPath queries, but the changes introduce several critical bugs.

  • Fixed HTML entity encoding display issues (but incorrectly changed the operator logic)
  • Reformatted long lines for better readability
  • Introduced incorrect file path references

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 17 comments.

File Description
reference/7.6/Microsoft.PowerShell.Diagnostics/Get-WinEvent.md Attempted to fix XPath operator encoding but changed logic from <= to >=, breaking the example; changed LogName to incorrect Logname casing
reference/7.5/Microsoft.PowerShell.Diagnostics/Get-WinEvent.md Same XPath operator issue; also introduced broken file paths by changing About folder to lowercase about and incorrectly lowercasing filenames; changed LogName to Logname; reformatted code indentation
reference/7.4/Microsoft.PowerShell.Diagnostics/Get-WinEvent.md XPath operator issue and line reformatting changes
reference/5.1/Microsoft.PowerShell.Diagnostics/Get-WinEvent.md XPath operator issue and line reformatting changes

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

Comment thread reference/7.6/Microsoft.PowerShell.Diagnostics/Get-WinEvent.md Outdated
Comment thread reference/7.5/Microsoft.PowerShell.Diagnostics/Get-WinEvent.md Outdated
Comment thread reference/7.4/Microsoft.PowerShell.Diagnostics/Get-WinEvent.md Outdated
Comment thread reference/7.5/Microsoft.PowerShell.Diagnostics/Get-WinEvent.md
Comment thread reference/7.5/Microsoft.PowerShell.Diagnostics/Get-WinEvent.md Outdated
Comment thread reference/7.5/Microsoft.PowerShell.Diagnostics/Get-WinEvent.md
Comment thread reference/7.5/Microsoft.PowerShell.Diagnostics/Get-WinEvent.md
Comment thread reference/7.6/Microsoft.PowerShell.Diagnostics/Get-WinEvent.md Outdated
Comment thread reference/7.4/Microsoft.PowerShell.Diagnostics/Get-WinEvent.md Outdated
Comment thread reference/5.1/Microsoft.PowerShell.Diagnostics/Get-WinEvent.md Outdated
@learn-build-service-prod
Copy link
Copy Markdown
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit cecedda:

✅ Validation status: passed

File Status Preview URL Details
reference/5.1/Microsoft.PowerShell.Diagnostics/Get-WinEvent.md ✅Succeeded View (powershell-5.1)
reference/7.4/Microsoft.PowerShell.Diagnostics/Get-WinEvent.md ✅Succeeded View (powershell-7.4)
reference/7.5/Microsoft.PowerShell.Diagnostics/Get-WinEvent.md ✅Succeeded View (powershell-7.5)
reference/7.6/Microsoft.PowerShell.Diagnostics/Get-WinEvent.md ✅Succeeded View (powershell-7.6)

For more details, please refer to the build report.

@sdwheeler sdwheeler merged commit 8da5bfe into MicrosoftDocs:main Jan 6, 2026
5 checks passed
@sdwheeler sdwheeler deleted the sdw-w543607-i12638-winevent branch January 12, 2026 15:33
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.

get-winevent -filterxpath example 16 doesn't work

2 participants