Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Conversation

@Dotnet-GitSync-Bot
Copy link
Contributor

@Dotnet-GitSync-Bot Dotnet-GitSync-Bot commented Apr 30, 2019

This PR contains mirrored changes from dotnet/coreclr,corert

Please REBASE this PR when merging

cc @tannergooding @krwq @marek-safar @buyaa-n @safern

@Anipik
Copy link

Anipik commented Apr 30, 2019

We may have to wait for decision on this one #37291 as the failure seems similar

@tannergooding
Copy link
Member

Just waiting on #37301 to be merged.

@krwq
Copy link
Member

krwq commented May 1, 2019

@Anipik - both seem to be merged, is there anything else blocking this?

@krwq krwq force-pushed the mirror-merge-10166303 branch from 7997436 to ef81aaa Compare May 1, 2019 19:48
tannergooding and others added 9 commits May 1, 2019 13:32
…e. (#23827)

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
* Nullable: System.Diagnostics.Tracing

* apply feedback

* fix one more comment

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
System.Type, System.Reflection nullability

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
* Nullable: Comparers, Dictionary and Friends

* Add object constraint to Dictionary

* Fix warning from new compiler and annotating dictionary

* PR Feedback

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
@krwq krwq force-pushed the mirror-merge-10166303 branch from c7105a7 to 8bcf3b8 Compare May 1, 2019 20:34
<GenFacadesIgnoreBuildAndRevisionMismatch>true</GenFacadesIgnoreBuildAndRevisionMismatch>

<!-- Warnings -->
<NoWarn>$(NoWarn);BCL0020</NoWarn> <!-- TODO: Remove BCL0020 once https://github.com/dotnet/arcade/pull/2158 is consumed. -->
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@stephentoub, this looks like it has likely been consumed by now.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the reminder. I'll take care of it.

@safern safern merged commit 07fdefa into dotnet:master May 1, 2019
@karelz karelz added this to the 3.0 milestone May 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants