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

Empty statement blocks have extraneous empty line #1039

Open
1 task done
verytactical opened this issue Nov 13, 2024 · 0 comments
Open
1 task done

Empty statement blocks have extraneous empty line #1039

verytactical opened this issue Nov 13, 2024 · 0 comments
Assignees
Labels
bug Something isn't working or isn't right scope: pretty-printer
Milestone

Comments

@verytactical
Copy link
Contributor

Are you using the latest released (or pre-released, a.k.a. "next") version?

  • I'm using the latest Tact version

Tact source code

init() {

}

Relevant Tact/build system log output

No response

What happened?

Pretty-printer keeps it as

init() {

}

What did you expect?

This doesn't look like expected behavior, because it's almost never formatted this way, and also requires explicit support for displaying empty statement blocks.

Unfortunately it's also persisted in our tests, and fixing it will break indentation in projects that already depend on pretty-printer's output.

Steps to reproduce

No response

How do you run Tact?

No response

Anything else?

No response

@verytactical verytactical added the bug Something isn't working or isn't right label Nov 13, 2024
@anton-trunov anton-trunov added this to the v1.6.0 milestone Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working or isn't right scope: pretty-printer
Projects
None yet
Development

No branches or pull requests

2 participants