-
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
Fix parameter completion for invalid scripts #17687
base: master
Are you sure you want to change the base?
Fix parameter completion for invalid scripts #17687
Conversation
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) |
Looking at the test failures, I guess we can't just ignore those errors in every scenario. I assumed I could because Experimental and hidden param attributes are ignored. |
WG discussion tag. On the one hand it makes sense to have more freedom with |
This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days. |
} | ||
catch (RuntimeException) | ||
{ | ||
continue; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since it is Compiler can this change a behavior of real script execution?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know. We need someone that is more familiar with the engine to answer that question.
@MartinGC94 Sorry for taking so long to review. Can you please resolve the conflicts for this PR? |
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
Fixes #4549
Fixes #12079
Fixes #11422
Allows tab completion for parameters to continue working when script requirements like
#requires -RunAsAdministrator
fail due to the current session not meeting the script requirements.Also allows tab completion to work for commands with parameters that use types that haven't been loaded yet, in such cases the type will be displayed as if the type hadn't been specified.
PR Context
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).