Skip to content

CodeDom null annotations fixes - #90393

Closed
halgab wants to merge 3 commits into
dotnet:mainfrom
halgab:codedom-null-winforms
Closed

CodeDom null annotations fixes#90393
halgab wants to merge 3 commits into
dotnet:mainfrom
halgab:codedom-null-winforms

Conversation

@halgab

Copy link
Copy Markdown
Contributor

This is a first step towards fixing #78036. Here, I fix three classes identified to have false annotations today. I hope to get some feedback from the team to see if I'm heading in the right direction.

@ghostghost added area-System.CodeDom community-contribution Indicates that the PR has been added by a community member labels Aug 11, 2023
@ghost

Copy link
Copy Markdown

Tagging subscribers to this area: @dotnet/area-system-codedom
See info in area-owners.md if you want to be subscribed.

Issue Details

This is a first step towards fixing #78036. Here, I fix three classes identified to have false annotations today. I hope to get some feedback from the team to see if I'm heading in the right direction.

Author:halgab
Assignees:-
Labels:

area-System.CodeDom, community-contribution

Milestone:-

@steveharter

Copy link
Copy Markdown
Contributor

Closing since #90401 disabled nullability for System.CodeDom. We can work on a full nullability approach in v9.

@ghostghost locked as resolved and limited conversation to collaborators Sep 13, 2023
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-System.CodeDomcommunity-contributionIndicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@halgab@steveharter