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

block comment in Rational between lparen and numerator is moved before lparen #2930

Closed
1 of 4 tasks
dawedawe opened this issue Jul 25, 2023 · 0 comments · Fixed by #2949
Closed
1 of 4 tasks

block comment in Rational between lparen and numerator is moved before lparen #2930

dawedawe opened this issue Jul 25, 2023 · 0 comments · Fixed by #2949
Assignees

Comments

@dawedawe
Copy link
Member

dawedawe commented Jul 25, 2023

Issue created from fantomas-online

Code

234<kg^((* foo *)2/3)>

Result

234<kg^ (* foo *) (2/3)>

Problem description

Please describe here the Fantomas problem you encountered.
Check out our Contribution Guidelines.

Extra information

  • The formatted result breaks my code.
  • The formatted result gives compiler warnings.
  • I or my company would be willing to help fix this.
  • I would like a release if this problem is solved.

Options

Fantomas main branch at 2023-07-24T13:06:38Z - 0e07cab

Default Fantomas configuration

Did you know that you can ignore files when formatting by using a .fantomasignore file?
PS: It's unlikely that someone else will solve your specific issue, as it's something that you have a personal stake in.

@dawedawe dawedawe self-assigned this Jul 25, 2023
@dawedawe dawedawe changed the title block comment in Rational between lparen and numerator is lost block comment in Rational between lparen and numerator is moved before lparen Aug 25, 2023
dawedawe added a commit to dawedawe/fantomas that referenced this issue Aug 25, 2023
…e parens and the div operator as real children of the RationalNode.

Fixes fsprojects#2930, fsprojects#2932, fsprojects#2933
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant