-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Browser Implementation For various assemblies #38559
Comments
I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label. |
Tagging subscribers to this area: @directhex |
Tagging subscribers to this area: @safern, @ViktorHofer |
Maybe we can omit everything that isn't changing. Only list the things we aim to change, so it should be a list that's 20-40 items. |
I did first pass and added comments on what will most likely need to change. I don't think we should hold original PR as there are/will be PRs (for example #37962) addressing this on per assembly basis as we enable more tests. |
Tagging subscribers to this area: @directhex |
Triage: @steveisok will update the table. We might want to add a column to track completeness. |
I pushed a rebased branch and github did autoclose. Probably because there is a commit that claims to be resolving this. |
I think we're in good shape here |
Closing as we are in good shape and any follow up will be handled on per assembly level |
Fixes dotnet/runtime#38559 Commit migrated from dotnet/runtime@5ce04cc
I will be filling the implementation details as the work goes on
@marek-safar @akoeplinger , i tried to summarize the implementation for each of the proj, can you guys take a look at the list and check if we need to make any changes ?
cc @ericstj
The text was updated successfully, but these errors were encountered: