-
Notifications
You must be signed in to change notification settings - Fork 0
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
Make line length configurable #83
base: main
Are you sure you want to change the base?
Conversation
Nice feature. Would be cool if you could also support configuration via pyproject.toml too. For some "inspiration" you can take a look at https://github.com/charliermarsh/ruff. It's a python linter written in rust. |
I tested, and at least for my style-feeling this looks much better.
come out as:
which I think doesn't look as good. Also:
with a long linelength will come out as:
|
In most cases I prefer to separate head and body that is why the line break after the
Which I think looks ugly too. I'm not happy with the feature .. while it allows to have multiple body atoms on one line, it also often disallows having them on separate lines. Maybe you are interested in the feature discussed here PR #86 |
You are right, I guess this is a game of preference. With the diffs, I agree that your method is superior. |
Hey @MaxOstrowski, this PR makes the maximal line length configurable.
clingofmt will read/create a
.clingofmt
file in TOML format, in this file you can defineline_length = 20
.Would be nice if you give it a test round. 💞