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

Apply ktlint with max_line_length as 112 #5456

Merged
merged 1 commit into from
Mar 12, 2024
Merged

Apply ktlint with max_line_length as 112 #5456

merged 1 commit into from
Mar 12, 2024

Conversation

sh-cho
Copy link
Contributor

@sh-cho sh-cho commented Feb 9, 2024

Motivation:

Currently kotlin maximum line length is set with 140 which is default for ktlint_official(https://pinterest.github.io/ktlint/latest/rules/standard/#max-line-length)

Modifications:

  • Set maximum line length of ktlint as 112. This rule is added by .editorconfig file.
  • Apply ktlint

Result:

@ikhoon ikhoon added the cleanup label Feb 13, 2024
@ikhoon ikhoon added this to the 1.28.0 milestone Feb 13, 2024
Copy link
Contributor

@ikhoon ikhoon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your contribution! 🙇‍♂️🙇‍♂️

Copy link
Member

@minwoox minwoox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thanks!

@minwoox minwoox merged commit df826c0 into line:main Mar 12, 2024
15 checks passed
@sh-cho sh-cho deleted the ktlint branch March 12, 2024 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Define ktlint rules
3 participants