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

自動的にscalafmtする #35

Merged
merged 12 commits into from
Mar 5, 2023
Merged

自動的にscalafmtする #35

merged 12 commits into from
Mar 5, 2023

Conversation

windymelt
Copy link
Owner

@windymelt windymelt commented Mar 5, 2023

solves: #

前提 / Prerequisites

  • scalafmtは導入されているが、自動的に適用はされていない

なぜこのPRが必要になったか / Why do we need this PR

  • 自動的に適用することで、コントリビュートによるコードスタイルの差分を吸収する
  • 一般に、この手のフォーマッタには忠実に従うほうが良いとされている

やったこと / What I did

  • scalafmtをsbtに入れる
  • scalafmtがCIで実行されるようにする

このPRが何を解決するか / What is solved by this PR

  • 自動でフォーマットされていないという状況

補足 / Supplementary information

場合によりフォーマットの設定は随時変更する。

@windymelt windymelt self-assigned this Mar 5, 2023
@windymelt windymelt merged commit 2e7c2c2 into main Mar 5, 2023
@windymelt windymelt deleted the automatic-scalafmt branch March 5, 2023 10:24
@xuwei-k
Copy link
Contributor

xuwei-k commented Jan 27, 2024

これだとおそらくforkしたrepoからのpull reqのjobで、checkout自体が失敗するから、もっと直さないとダメだと思います

@xuwei-k
Copy link
Contributor

xuwei-k commented Jan 27, 2024

これだとおそらくforkしたrepoからのpull reqのjobで、checkout自体が失敗するから、もっと直さないとダメだと思います

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

Successfully merging this pull request may close these issues.

2 participants