-
Notifications
You must be signed in to change notification settings - Fork 23
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
Enable containers 0.7 (ghc 9.10) #117
Conversation
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.
I guess it's better to remove the upper bound in this case.
I don't understand..? |
I mean, I can remove the upper bound if you like, or set it to something really high (otherwise Hackage will not accept the package at all), but it's not an automatic given that |
I have suffered from the major version up of |
Removing the upper bound altogether will cause Hackage to reject the package. Will I set it to |
I would like to remove the upper bound only for |
Oh, maybe you are right, I just checked, and it's a warning only...? 🤔 Only if it is |
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.
Now LGTM
Rebased and merged. |
No description provided.