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.

Sign up for GitHub

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

Jump to bottom

Ktlintの設定 #3

Merged
merged 3 commits into from Jun 24, 2021
Merged

Ktlintの設定 #3

merged 3 commits into from Jun 24, 2021

Conversation

mori-atsushi
Copy link
Owner

概要

  • ktlintの設定
  • gradle scriptに追加

確認してほしいこと

  • 素のktlintを呼び出しているが、他におすすめの方法あれば教えてほしいです

b4tchkn
b4tchkn approved these changes Jun 24, 2021
View reviewed changes
Copy link

@b4tchkn b4tchkn left a comment

Choose a reason for hiding this comment

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

僕もいつもこれ使ってるので、なにか他にあったら僕も気になります
普段の開発ではチームの方針によりますが、trailing commaを強制させるやつとかあるんですかね？

@mori-atsushi
Copy link
Owner Author

trailing commaはまだ入ってなさそうですかね？
カスタムルールとして追加する方法はありそうです。
pinterest/ktlint#709

@mori-atsushi mori-atsushi merged commit fd662cb into main Jun 24, 2021
@mori-atsushi mori-atsushi deleted the atsushi/ktlint branch June 24, 2021 02:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@b4tchkn b4tchkn b4tchkn approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@mori-atsushi @b4tchkn