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

Nullable: All remaining exceptions #23676

Merged
merged 2 commits into from
Apr 3, 2019

Conversation

safern
Copy link
Member

@safern safern commented Apr 2, 2019

@stephentoub I left out of this PR:
System\Threading\Tasks\TaskCanceledException.cs
System\Threading\Tasks\TaskExceptionHolder.cs
System\Threading\Tasks\TaskSchedulerException.cs

I didn't know if you were already working on them locally, I didn't want to duplicate work. If you're not, just let me know and I'll include them on my PR.

@safern safern requested a review from a team April 2, 2019 23:50
@stephentoub
Copy link
Member

I left out of this PR

Thanks, yes, I've got those edited locally.

Copy link
Member

@danmoseley danmoseley left a comment

Choose a reason for hiding this comment

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

With comments.

It's a shame none of these exceptions validate their messages aren't null. They needn't have followed Exception.

@safern safern merged commit cdcca03 into dotnet:NullableFeature Apr 3, 2019
@safern safern deleted the NullableMoreExceptions branch April 3, 2019 18:51
Dotnet-GitSync-Bot pushed a commit to Dotnet-GitSync-Bot/corefx that referenced this pull request Apr 5, 2019
* Nullable: All remaining exceptions

* Fix build break and PR Feedback

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
stephentoub pushed a commit to dotnet/corefx that referenced this pull request Apr 6, 2019
* Nullable: All remaining exceptions

* Fix build break and PR Feedback

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Dotnet-GitSync-Bot pushed a commit to Dotnet-GitSync-Bot/corert that referenced this pull request Apr 9, 2019
* Nullable: All remaining exceptions

* Fix build break and PR Feedback

Signed-off-by: dotnet-bot <anirudhagnihotry098@gmail.com>
Dotnet-GitSync-Bot pushed a commit to Dotnet-GitSync-Bot/corert that referenced this pull request Apr 9, 2019
* Nullable: All remaining exceptions

* Fix build break and PR Feedback

Signed-off-by: dotnet-bot <anirudhagnihotry098@gmail.com>
Dotnet-GitSync-Bot pushed a commit to Dotnet-GitSync-Bot/mono that referenced this pull request Apr 9, 2019
* Nullable: All remaining exceptions

* Fix build break and PR Feedback

Signed-off-by: dotnet-bot <anirudhagnihotry098@gmail.com>
Anipik pushed a commit to Dotnet-GitSync-Bot/corert that referenced this pull request Apr 9, 2019
* Nullable: All remaining exceptions

* Fix build break and PR Feedback

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
jkotas pushed a commit to dotnet/corert that referenced this pull request Apr 10, 2019
* Nullable: All remaining exceptions

* Fix build break and PR Feedback

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
* Nullable: All remaining exceptions

* Fix build break and PR Feedback


Commit migrated from dotnet/coreclr@cdcca03
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.

4 participants