Skip to content

Fixes #12803 - Add note about $null return for properties#12805

Merged
michaeltlombardi merged 2 commits intoMicrosoftDocs:mainfrom
sdwheeler:sdw-w557999-i12803-property-access
Mar 2, 2026
Merged

Fixes #12803 - Add note about $null return for properties#12805
michaeltlombardi merged 2 commits intoMicrosoftDocs:mainfrom
sdwheeler:sdw-w557999-i12803-property-access

Conversation

@sdwheeler
Copy link
Copy Markdown
Collaborator

@sdwheeler sdwheeler commented Mar 2, 2026

PR Summary

Add note about $null return for properties

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 March 2, 2026 20:33
@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.

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

Updates the about_Operators documentation to note that member access can return $null when properties are missing or when property getters throw, addressing the behavior reported in #12803 across multiple versioned docs.

Changes:

  • Add a new note in the Member-access operator . section describing $null results for missing/exceptional property access.
  • Update and renumber several link references in the affected documents.

Reviewed changes

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

File Description
reference/7.6/Microsoft.PowerShell.Core/About/about_Operators.md Adds $null-return note under member access; updates link reference numbering to accommodate new external link.
reference/7.5/Microsoft.PowerShell.Core/About/about_Operators.md Same content update as 7.6 for the member-access note and link reference renumbering.
reference/7.4/Microsoft.PowerShell.Core/About/about_Operators.md Same content update as 7.6 for the member-access note and link reference renumbering.
reference/5.1/Microsoft.PowerShell.Core/About/about_Operators.md Same member-access note added (with corresponding link reference renumbering for 5.1 doc set).

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

Comment thread reference/7.6/Microsoft.PowerShell.Core/About/about_Operators.md Outdated
Comment thread reference/7.5/Microsoft.PowerShell.Core/About/about_Operators.md Outdated
Comment thread reference/7.4/Microsoft.PowerShell.Core/About/about_Operators.md Outdated
Comment thread reference/5.1/Microsoft.PowerShell.Core/About/about_Operators.md Outdated
@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 569b4db:

✅ Validation status: passed

File Status Preview URL Details
reference/5.1/Microsoft.PowerShell.Core/About/about_Operators.md ✅Succeeded View (powershell-5.1)
reference/7.4/Microsoft.PowerShell.Core/About/about_Operators.md ✅Succeeded View (powershell-7.4)
reference/7.5/Microsoft.PowerShell.Core/About/about_Operators.md ✅Succeeded View (powershell-7.5)
reference/7.6/Microsoft.PowerShell.Core/About/about_Operators.md ✅Succeeded View (powershell-7.6)

For more details, please refer to the build report.

@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 9e0342e:

✅ Validation status: passed

File Status Preview URL Details
reference/5.1/Microsoft.PowerShell.Core/About/about_Operators.md ✅Succeeded View (powershell-5.1)
reference/7.4/Microsoft.PowerShell.Core/About/about_Operators.md ✅Succeeded View (powershell-7.4)
reference/7.5/Microsoft.PowerShell.Core/About/about_Operators.md ✅Succeeded View (powershell-7.5)
reference/7.6/Microsoft.PowerShell.Core/About/about_Operators.md ✅Succeeded View (powershell-7.6)

For more details, please refer to the build report.

@michaeltlombardi michaeltlombardi merged commit fb8e8d1 into MicrosoftDocs:main Mar 2, 2026
6 checks passed
@sdwheeler sdwheeler deleted the sdw-w557999-i12803-property-access branch March 4, 2026 21:50
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.

Document that exceptions from property getters are always ignored, return $null

3 participants