-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
[release/8.0] Fix Type.ContainsGenericParameters for function pointers #90963
Conversation
Tagging subscribers to this area: @dotnet/area-system-reflection Issue DetailsBackport of #90864 to release/8.0 /cc @jkotas Customer ImpactTestingRiskIMPORTANT: If this backport is for a servicing release, please verify that:
|
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.
approved. please get a code review and when ready this can be merged
@ivanpovazan @steveharter can you please review and sign-off? |
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.
Approving as Steve is out, the PR has no difference from #90864
Backport of #90864 to release/8.0
/cc @jkotas
Customer Impact
Missing functionality for new feature added in 8.0.
Testing
Unit tests
Risk
IMPORTANT: If this backport is for a servicing release, please verify that:
The PR target branch is
release/X.0-staging
, notrelease/X.0
.If the change touches code that ships in a NuGet package, you have added the necessary package authoring and gotten it explicitly reviewed.