-
Notifications
You must be signed in to change notification settings - Fork 7.7k
Add support for interface static abstract meths #21062
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
base: master
Are you sure you want to change the base?
Add support for interface static abstract meths #21062
Conversation
286d581
to
08fc5b3
Compare
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) |
@jborean93 I need to investigate #21253 and a few other issues related the WDAC Audit feature, to see if we can fix the root causes for the v7.4.2 release. I will review your PowerShell Class PRs next Monday. |
Thanks for the update! |
@jborean93 Please resolve merge conflicts. |
08fc5b3
to
30998a3
Compare
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) |
Done! |
30998a3
to
5856b5d
Compare
5856b5d
to
ed0606a
Compare
ed0606a
to
869d10c
Compare
869d10c
to
42dd8e3
Compare
Add support for writing a PowerShell class that can implement an interface that contains a static abstract method introduced in C# 11.
42dd8e3
to
78b7635
Compare
PR Summary
Add support for writing a PowerShell class that can implement an interface that contains a static abstract method introduced in C# 11.
I'm not too happy with the implementation done here for matching against the
Type[]
values. If you can think off a better/more performant way to do this then I'm happy to change it.PR Context
Fixes: #21060
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).