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

Grainfmt: Experiment with removal of infix_wrap argument #2036

Open
phated opened this issue Feb 19, 2024 · 1 comment
Open

Grainfmt: Experiment with removal of infix_wrap argument #2036

phated opened this issue Feb 19, 2024 · 1 comment

Comments

@phated
Copy link
Member

phated commented Feb 19, 2024

The infix_wrap argument mostly seems like a workaround and I think we could better factor out the behavior. This is just cleanup on #1976 so we can do it later.

@ospencer
Copy link
Member

ospencer commented Apr 5, 2024

An idea would be to create a print_infix_expression function which applies the wrapping, and call it from everywhere we pass ~infix_wrap=Fun.id.

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

No branches or pull requests

2 participants