Skip to content

Fixes 6431 - Updates about_Splatting - #6473

Merged
David Coulter (DCtheGeek) merged 8 commits into
MicrosoftDocs:stagingfrom
chasewilson:chasewilson-ghi6431
Aug 12, 2020
Merged

Fixes 6431 - Updates about_Splatting#6473
David Coulter (DCtheGeek) merged 8 commits into
MicrosoftDocs:stagingfrom
chasewilson:chasewilson-ghi6431

Conversation

@chasewilson

@chasewilson Chase Wilson (chasewilson) commented Aug 11, 2020 •
edited by DCtheGeek
Loading

Copy link
Copy Markdown
Contributor

PR Summary

New behavior for splatting in PowerShell 7.1. You can now supercede splatted parameters by explicitly defining a param in the calling command.

PR Context

Fixes #6431
Fixes AB#1758173

Select the area of the Table of Contents containing the documents being changed.

Conceptual content

  • Overview and Install
  • Learning PowerShell
    • PowerShell 101
    • Deep dives
    • Remoting
  • Release notes (What's New)
  • Windows PowerShell
    • WMF, ISE, release notes, etc.
  • DSC articles
  • Community resources
  • Sample scripts
  • Gallery articles
  • Scripting and development
    • Legacy SDK

Cmdlet reference & about_ topics

  • Version 7.1 preview content
  • Version 7.0 content
  • Version 6 content
  • Version 5.1 content

PR Checklist

  • I have read the contributors guide and followed the style and process guidelines
  • PR has a meaningful title
  • PR is targeted at the staging branch
  • All relevant versions updated
  • Includes content related to issues and PRs - see Closing issues using keywords.
  • This PR is ready to merge and is not Work in Progress
    • If the PR is work in progress, please add the prefix WIP: or [WIP] to the beginning of the
      title and remove the prefix when the PR is ready.

@opbld30

opbld30 commented Aug 11, 2020

Copy link
Copy Markdown

Docs Build status updates of commit 880afe1:

✅ Validation status: passed

File Status Preview URL Details
.vscode/settings.json ✅Succeeded
reference/7.1/Microsoft.PowerShell.Core/About/about_Splatting.md ✅Succeeded View (powershell-7.1)
reference/docs-conceptual/learn/deep-dives/everything-about-error-handling.md ✅Succeeded View (powershell-5.1)
View (powershell-6)
View (powershell-7)
View (powershell-7.1)

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@opbld30

opbld30 commented Aug 11, 2020

Copy link
Copy Markdown

Docs Build status updates of commit c4b5aeb:

✅ Validation status: passed

File Status Preview URL Details
.vscode/settings.json ✅Succeeded
reference/7.1/Microsoft.PowerShell.Core/About/about_Splatting.md ✅Succeeded View (powershell-7.1)
reference/docs-conceptual/learn/deep-dives/everything-about-error-handling.md ✅Succeeded View (powershell-5.1)
View (powershell-6)
View (powershell-7)
View (powershell-7.1)

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

sdwheeler
Sean Wheeler (sdwheeler) suggested changes Aug 12, 2020
View reviewed changes

@sdwheeler Sean Wheeler (sdwheeler) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think the new example will work. Just need to clean up a few items.

Comment thread .vscode/settings.json Resolved
Comment thread reference/7.1/Microsoft.PowerShell.Core/About/about_Splatting.md Resolved
Comment thread reference/7.1/Microsoft.PowerShell.Core/About/about_Splatting.md Resolved
Comment thread reference/docs-conceptual/learn/deep-dives/everything-about-error-handling.md Resolved
@chasewilson

Chase Wilson (chasewilson) commented Aug 12, 2020

Copy link
Copy Markdown
Contributor Author

Requested changes are in place. #sign-off

@opbld31

opbld31 commented Aug 12, 2020

Copy link
Copy Markdown

Docs Build status updates of commit 65a3b11:

✅ Validation status: passed

File Status Preview URL Details
.gitignore ✅Succeeded
reference/7.1/Microsoft.PowerShell.Core/About/about_Splatting.md ✅Succeeded View (powershell-7.1)

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@chasewilson Chase Wilson (chasewilson) changed the title [WIP]Fixes 6431 - Updates about_Splatting Fixes 6431 - Updates about_Splatting Aug 12, 2020
DCtheGeek
David Coulter (DCtheGeek) suggested changes Aug 12, 2020
View reviewed changes
Comment thread reference/7.1/Microsoft.PowerShell.Core/About/about_Splatting.md Resolved
Comment thread reference/7.1/Microsoft.PowerShell.Core/About/about_Splatting.md Resolved
Comment thread reference/7.1/Microsoft.PowerShell.Core/About/about_Splatting.md Resolved
Comment thread reference/7.1/Microsoft.PowerShell.Core/About/about_Splatting.md Resolved
Comment thread reference/7.1/Microsoft.PowerShell.Core/About/about_Splatting.md Resolved
@opbld31

opbld31 commented Aug 12, 2020

Copy link
Copy Markdown

Docs Build status updates of commit 8ca1b7c:

✅ Validation status: passed

File Status Preview URL Details
.gitignore ✅Succeeded
reference/7.1/Microsoft.PowerShell.Core/About/about_Splatting.md ✅Succeeded View (powershell-7.1)

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@DCtheGeek
David Coulter (DCtheGeek) merged commit 5810718 into MicrosoftDocs:staging Aug 12, 2020
@chasewilson
Chase Wilson (chasewilson) deleted the chasewilson-ghi6431 branch December 17, 2020 22:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers
@sdwheeler Sean Wheeler (sdwheeler) sdwheeler requested changes
@DCtheGeek David Coulter (DCtheGeek) DCtheGeek approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Explicitly specified named arguments will supersede the same ones from a Hashtable splatting

5 participants

@chasewilson @opbld30 @opbld31 @DCtheGeek @sdwheeler