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

Ugly re-formatted code #444

Closed
lburgazzoli opened this issue Nov 20, 2019 · 4 comments
Closed

Ugly re-formatted code #444

lburgazzoli opened this issue Nov 20, 2019 · 4 comments

Comments

@lburgazzoli
Copy link
Contributor

What I wrote:

image

The same code after the auto formatter plugin kicks in:

image

I know that I can add CHECKSTYLE:OFF|ON but it would be nice if we can check if there are any better code style combo that won't break the code so ugly

@ppalaga
Copy link
Contributor

ppalaga commented Nov 20, 2019

Adding a newline before the post() statement would perhaps improve the readability of the formatted result.

For the record: This is not a problem for me.

@lburgazzoli
Copy link
Contributor Author

Another case is given/then notation with RestAssured

what I'd like:
image

what the formatter does:
image

@ppalaga
Copy link
Contributor

ppalaga commented May 26, 2021

Do we still want to do something about this?

@lburgazzoli
Copy link
Contributor Author

I think something has bee put in place in the format plugin so we can close this

@ppalaga ppalaga closed this as completed May 26, 2021
@ppalaga ppalaga added this to the No fix/wont't fix milestone May 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants