-
Notifications
You must be signed in to change notification settings - Fork 8.1k
Add ExcludeModule parameter to Get-Command #18955
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
Add ExcludeModule parameter to Get-Command #18955
Conversation
|
The parameter name should be |
|
This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days. |
|
Queued this up for the WG-Cmdlets to discuss |
|
The Cmdlet WG reviewed this and agree with adding this parameter to Get-Command. We're not making any statement on the quality of this PR. |
|
This PR has Quantification details
Why proper sizing of changes matters
Optimal pull request sizes drive a better predictable PR flow as they strike a
What can I do to optimize my changes
How to interpret the change counts in git diff output
Was this comment helpful? 👍 :ok_hand: :thumbsdown: (Email) |
|
This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days. |
|
What is the status of this PR? |
|
Hi all. What is the status of this PR? |
|
Hi all again. Just following up. What is the status of this PR? |
|
Sorry to ask again, but what is the status of this PR @iSazonov and @daxian-dbw? |
|
@alanlivio Good news, according to the PS team blogpost: https://devblogs.microsoft.com/powershell/powershell-and-openssh-team-investments-for-2024/ one of the goals for 2024 is to focus on getting community PRs merged. Obviously they haven't started doing that yet, but it would be very weird if they didn't start within the next couple of weeks after making a blog post like that.
|
|
Good news. I hope that they can merge your PR soon. |
test/powershell/Modules/Microsoft.PowerShell.Core/Get-Command.Tests.ps1
Outdated
Show resolved
Hide resolved
|
This was approved but not merged. Are we waiting for an official 7.5 release before merging or what's the idea here? |
|
Gonna close and reopen to get CI going, once it passes I'll get it merged |
|
@SeeminglyScience The PR is too old - I suggest to rebase before merge. |
|
I am looking forward to seeing this merged. |
f0c9d6e to
46e0678
Compare
PR Summary
Adds the ExcludeModule parameter to Get-Command so you can exclude commands from specified modules like this:
Get-Command *disk* -ExcludeModule Storage.PR Context
The main purpose is to contribute towards: #16747 but like the previous example shows it can also be useful when manually searching for commands.
PR Checklist
.h,.cpp,.cs,.ps1and.psm1files have the correct copyright headerWIP:or[ WIP ]to the beginning of the title (theWIPbot will keep its status check atPendingwhile the prefix is present) and remove the prefix when the PR is ready.-ExcludeModuleparameter forGet-Commandcmdlet MicrosoftDocs/PowerShell-Docs#11356(which runs in a different PS Host).