Closed simschla closed 1 year ago
Thanks for your interest in palantir/palantir-java-format, @simschla! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.
Type
Description
@fawind looking for maintainer's insights here - is this something palantir-java-format
would be open to doing? (It is clearly a wish on the customer side and seems pretty easy to achieve.)
Any updates on this? @simschla thanks for the pr by the way
Thanks for putting up this PR! I'm sorry, but for now, I don't think we want to pursue making this configurable. I left a comment on the origin issue with some thoughts: https://github.com/palantir/palantir-java-format/issues/859#issuecomment-1665671443.
Thanks for putting up this PR! I'm sorry, but for now, I don't think we want to pursue making this configurable. I left a comment on the origin issue with some thoughts: #859 (comment).
There are any specific reason to deny this PR ? Is pretty basic to be able to change line character limit, a lot of developer we are using +2K screens.
Before this PR
The max line length for Palantir is 120 characters.
After this PR
The max line length for Palantir java format is configurable. ==COMMIT_MSG== Enable users to override max line length. ==COMMIT_MSG==
Possible downsides?
We are introducing configurability, which diverts from the way GJF handles things.
Notes
com.palantir.javaformat.java.FormatterService
for allowing configuration. I'm open for suggestions here.This resolves #859