-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Add NotNull attributes to query code #16705
Labels
area-query
closed-fixed
The issue has been fixed and is/will be included in the release indicated by the issue milestone.
customer-reported
type-cleanup
Milestone
Comments
ajcvickers
added
the
closed-fixed
The issue has been fixed and is/will be included in the release indicated by the issue milestone.
label
Dec 1, 2019
ajcvickers
added a commit
that referenced
this issue
Dec 1, 2019
Fixes #16705 Attempted to add null checks when the API is a public-surface API, but not for purely internal APIs. There may be some mistakes here; I plan to do a bit more static analysis after the initial merge. Also fixed the ApiConsistencyTest to check pubternal types again (not sure why this was removed) and to check protected constructors.
ajcvickers
added a commit
that referenced
this issue
Dec 2, 2019
Fixes #16705 Attempted to add null checks when the API is a public-surface API, but not for purely internal APIs. There may be some mistakes here; I plan to do a bit more static analysis after the initial merge. Also fixed the ApiConsistencyTest to check pubternal types again (not sure why this was removed) and to check protected constructors.
ajcvickers
added a commit
that referenced
this issue
Dec 8, 2019
Fixes #16705 Attempted to add null checks when the API is a public-surface API, but not for purely internal APIs. There may be some mistakes here; I plan to do a bit more static analysis after the initial merge. Also fixed the ApiConsistencyTest to check pubternal types again (not sure why this was removed) and to check protected constructors.
ajcvickers
added a commit
that referenced
this issue
Dec 8, 2019
Fixes #16705 Attempted to add null checks when the API is a public-surface API, but not for purely internal APIs. There may be some mistakes here; I plan to do a bit more static analysis after the initial merge. Also fixed the ApiConsistencyTest to check pubternal types again (not sure why this was removed) and to check protected constructors.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
area-query
closed-fixed
The issue has been fixed and is/will be included in the release indicated by the issue milestone.
customer-reported
type-cleanup
Breaking out from #15796 since it's not breaking to add these later.
The text was updated successfully, but these errors were encountered: