-
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
Set GenFacadesForceZeroVersionSeeds for all shims #33052
Conversation
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.
@ericstj are we porting this to 3.1 as well ?
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.
Changes LGTM. I'm sure you did so already, but it would be good to do at least one ILDiff of one of our shims and maybe compare it with our 2.x ones, to make sure that we actually match with those this time 😄
I did diff this when I made the arcade change. I'll rebuild and share a diff with the latest. Looks like APICompat isn't happy with the new shims, I may need to investigate that. |
Here's the diffs of mscorlib between 2.1 -> latest and 3.1 -> latest. |
1403442
to
819853c
Compare
Hello @ericstj! Because this pull request has the p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (
|
First I'd like to let this ship in a preview in 5.0, once we have confirmed it's ok we can port to servicing. |
Opts into GenFacadesForceZeroVersionSeeds for all shims. This is consistent with what we did in 2.x.
I've manually updated GenFacades rather than wait for #32858