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

Temporarily revert #1720 #1733

Merged
merged 1 commit into from
Jan 6, 2018
Merged

Temporarily revert #1720 #1733

merged 1 commit into from
Jan 6, 2018

Conversation

chenglou
Copy link
Member

@chenglou chenglou commented Jan 6, 2018

See #1720 (comment)

This way of printing with parens while parsing both lets userland
gradually use the enforced parens syntax without breaking their app
(especially, and mainly, if they have third-party dependencies).

We'll re-revert this patch in the future.

See #1720 (comment)

This way of printing with parens while parsing both lets userland
gradually use the enforced parens syntax without breaking their app
(especially, and mainly, if they have third-party dependencies).

We'll re-revert this patch in the future.
@chenglou chenglou merged commit 5457093 into master Jan 6, 2018
@chenglou chenglou deleted the foo.re branch January 6, 2018 13:17
@chenglou
Copy link
Member Author

chenglou commented Jan 6, 2018

cc @let-def.

Feel free to re-revert this diff (aka basically build on top of 1720) for your experiment on the percent-less extension point syntax

@let-def
Copy link
Contributor

let-def commented Jan 6, 2018

Thanks! :)

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

Successfully merging this pull request may close these issues.

3 participants