-
Notifications
You must be signed in to change notification settings - Fork 1.1k
fix #13332: change accessibility in whyNotGenericSum #13393
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
Conversation
@bishabosha tests failed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't tell if this is allowing false negatives, but it looks like a reasonable change to make the case work.
I will add more test cases |
to solve #13332 needs more than an accessibility test, it also affects code generation as the prefix of the outer class needs to be substituted wherever it can appear, see comment #13332 (comment) I will see what I can do today |
I don't think this can be solved before Monday |
I'll see what I can do today - but I imagine reviewing would not meet the deadline, so perhaps I should remove again |
Even if it could be reviewed it should wait IMO. Big changes in functionality should be tested in nightlies for an extended period of time before being rolled into a release. |
closing this in favor of a new approach |
change to use
isAccessibleFrom
fixes #13332