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

Is it possible to tweak :remove-consecutive-blank-lines? so that it accepts 3 or 2 blank lines instead of only 2? #275

Open
pdelfino opened this issue Oct 25, 2022 · 1 comment

Comments

@pdelfino
Copy link

Hi,

Congrats on the awesome work! I am planning to use this plug-in on the project that I am working on at my job.

Currently, the documentation says that:

:remove-consecutive-blank-lines? - true if cljfmt should collapse consecutive blank lines. This will convert (foo)\n\n\n(bar) to (foo)\n\n(bar). Defaults to true.

Unfortunately, one of the developers like to use 3 blank lines to express a particular meaning.

Would it be possible to tweak the tool so that it accepts 2 or 3 blank lines instead of only 2?

Thank you!

@weavejester
Copy link
Owner

Possibly... I'm open to it being an option, but since this is the first time I've been asked about this to my recollection, this may be too niche a feature. Have you considered managing this with a separate regular expression? e.g. (str/replace s #"\n{4,}" "\n\n\n").

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

No branches or pull requests

2 participants