Skip to content

Conversation

@aosen-xiong
Copy link
Member

No description provided.

Copy link
Member

@wmdietl wmdietl left a comment

Choose a reason for hiding this comment

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

Thanks, nice cleanup!
Will you enable the check for redundant annotations in eisop/checker-framework#1145 ? Or how did you find the redundancies?

@aosen-xiong
Copy link
Member Author

aosen-xiong commented May 13, 2025

Maybe after I fixed all the type check errors. I found those Nonnull when I try to fix the type check errors in WeakPairMap.

@wmdietl
Copy link
Member

wmdietl commented May 13, 2025

Should we wait until this check is automated, to make sure all of these are correct? They look fine, but the automated check would be even nicer and possibly more complete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants