-
Notifications
You must be signed in to change notification settings - Fork 667
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 option Typeclasses Default Generalize Instance Variables. #6319
Conversation
Not sure the \ref in the documentation is the correct one.
You should probably turn this option off in the compatibility file |
No, the default behaviour is the 8.7 behaviour. |
Thanks! IIRC 8.7 does not generalize typeclass instance variables per default, but only if Also, what is the interaction of the new option with |
It only generalizes those variables which are generalizable.
which enables it and without an explicit marker the option decides which is picked (with 8.7 always picking `) Github using ` for code blocks is a pain. |
But that changes behavior, right? The following currently passes:
With your PR, because it defaults to |
It already defaults to |
Ah, so |
It's probably more complicated than that. Like what is going on here 95dd730 |
If I come back to this issue I probably won't base my work on this PR. |
I opened an issue at #6888 to make sure this doesn't get forgotten. |
Not sure the \ref in the documentation is the correct one.
Not sure I completely understood what it's doing, as I feel like the meaning of
Implicit
andExplicit
are inverted.Ping @RalfJung, see #6030.