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

define maximum number of empty lines #164

Open
stigh opened this issue Feb 19, 2024 · 0 comments · May be fixed by #165
Open

define maximum number of empty lines #164

stigh opened this issue Feb 19, 2024 · 0 comments · May be fixed by #165

Comments

@stigh
Copy link

stigh commented Feb 19, 2024

Today fprettify will not allow more than 1 consecutive empty lines. Personally I like to keep 2 empty lines between subroutines and functions for improved readability.

See inspiration from eslint here: https://eslint.style/rules/js/no-multiple-empty-lines. According to eslint documentation:

Some developers prefer to have multiple blank lines removed, while others feel that it helps improve readability. Whitespace is useful for separating logical sections of code, but excess whitespace takes up more of the screen.

I see there is a PR in #106, which seem to have been closed unintentionally. --keep-blank-lines would solve my issue, but I think an even better solution would be to define a maximum value.

@stigh stigh linked a pull request Feb 28, 2024 that will close this issue
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

Successfully merging a pull request may close this issue.

1 participant