- Notifications
You must be signed in to change notification settings - Fork7.7k
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
The parameter name should be |
ghost commentedJan 25, 2023
This pull request has been automatically marked as Review Needed because it has been there has not been any activity for7 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. |
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
This PR has Quantification details
Why proper sizing of changes mattersOptimal 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) |
ghost commentedFeb 11, 2023
This pull request has been automatically marked as Review Needed because it has been there has not been any activity for7 days. |
What is the status of this PR? |
alanlivio commentedNov 24, 2023
Hi all. What is the status of this PR? |
alanlivio commentedJan 3, 2024
Hi all again. Just following up. What is the status of this PR? |
alanlivio commentedJan 26, 2024
Sorry to ask again, but what is the status of this PR@iSazonov and@daxian-dbw? |
MartinGC94 commentedFeb 7, 2024 • edited by unfurl-linksbot
Loading Uh oh!
There was an error while loading.Please reload this page.
edited by unfurl-linksbot
Uh oh!
There was an error while loading.Please reload this page.
@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.
|
alanlivio commentedFeb 7, 2024
Good news. I hope that they can merge your PR soon. |
Uh oh!
There was an error while loading.Please reload this page.
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. |
alanlivio commentedDec 12, 2024
I am looking forward to seeing this merged. |
f0c9d6e
to46e0678
Comparec066cd8
intoPowerShell:masterUh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
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.-ExcludeModule
parameter forGet-Command
cmdlet MicrosoftDocs/PowerShell-Docs#11356(which runs in a different PS Host).