Skip to content
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

Remove dot from characters allowed in overloaded labels #573

Conversation

adamgundry
Copy link
Contributor

This is necessary to avoid breaking existing code using the (#.) operator. See discussion at #170 (comment).

cc @howtonotwin @goldfirere

This is necessary to avoid breaking existing code using the `(#.)` operator.
@int-index
Copy link
Contributor

LGTM

@simonpj
Copy link
Contributor

simonpj commented Jan 26, 2023

I'm strongly supportive of fixing this. I don't feel close enough to have a well-informed opinion about the details of the fix.

Thanks to Adam for proposing something concrete. +1 from me.

@adamgundry
Copy link
Contributor Author

@nomeata This change is urgent to allow GHC 9.6 to be released without causing breaking changes. I don't know how much of the full process it needs to go through, but I'd like to submit to the committee.

@nomeata nomeata self-assigned this Feb 1, 2023
@nomeata nomeata added Pending committee review The committee needs to evaluate the proposal and make a decision Accepted The committee has decided to accept the proposal and removed Pending committee review The committee needs to evaluate the proposal and make a decision labels Feb 1, 2023
@nomeata nomeata merged commit b76b143 into ghc-proposals:master Feb 5, 2023
@adamgundry adamgundry added the Implemented The proposal has been implemented and has hit GHC master label Feb 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted The committee has decided to accept the proposal Implemented The proposal has been implemented and has hit GHC master
Development

Successfully merging this pull request may close these issues.

None yet

4 participants