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

Use rustfmt when translating #742

Open
LegNeato opened this issue Nov 27, 2022 · 1 comment
Open

Use rustfmt when translating #742

LegNeato opened this issue Nov 27, 2022 · 1 comment
Labels
enhancement New feature or request

Comments

@LegNeato
Copy link
Contributor

rustfmt used to be part of the refactoring tool but currently isn't part of refactoring or translation. I believe that formatting with rustfmt should be part of the translation tool. Doing so will make reading the generated code nicer, makes it so the transpiler code doesn't ever need to care about formatting, and matches user expectations.

@kkysen
Copy link
Contributor

kkysen commented Nov 28, 2022

I agree. Given the refactoring engine is not currently supported, and running rustfmt on the transpiler output is quite easy, I think this is something we should do. That said, it is quite easy to do yourself, too.

@kkysen kkysen added the enhancement New feature or request label Nov 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants