-
Notifications
You must be signed in to change notification settings - Fork 7.1k
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
base: master
Are you sure you want to change the base?
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. |
This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days. |
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
,.ps1
and.psm1
files have the correct copyright headerWIP:
or[ WIP ]
to the beginning of the title (theWIP
bot will keep its status check atPending
while the prefix is present) and remove the prefix when the PR is ready.(which runs in a different PS Host).