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

全員共通のコードフォーマットの設定を追加したい #21

Open
Elise3993 opened this issue Jul 19, 2024 · 8 comments
Open
Assignees
Labels
enhancement New feature or request

Comments

@Elise3993
Copy link
Collaborator

Elise3993 commented Jul 19, 2024

概要

  • このプロジェクトに参加しているメンバー共通の、コードフォーマット設定を追加したい
  • VSかVSCodeかとかのエディタの違い、人によるコーディングの好みとかを吸収して共通のコーディング規約を設定したい

手動でフォーマットすることの問題点

  • コード規約や綺麗さを、人の手によって守ろうとすると色々不都合が起きる
    • コーディング規約を全員で決めてもヒューマンエラーが起き得る
    • コーディング規約を厳密に考えながらコーディングするのは時間がかかる
    • 規約で厳密に決めないようなルール、例えば行末の改行やスペースの数、インデントがタブorスペースとかが、人によって変わるとコミットでのDiffがちょっと見づらくなる

方法

ざっと調べた限りでは2つほど出てきた。手間が少なそうなので、EditorConfigの方を採用したいと思ってます

@Elise3993 Elise3993 added the enhancement New feature or request label Jul 19, 2024
@Elise3993 Elise3993 self-assigned this Jul 19, 2024
@Elise3993
Copy link
Collaborator Author

どうも無さそうではあったのですが、「不要な改行を詰める」みたいな設定オプションが設定できると大変うれしいので、見つけた人は教えてもらえると嬉しいです。

@AiletS
Copy link
Collaborator

AiletS commented Jul 21, 2024

editorconfig.txt
テンプレはこれ使ってました

@Elise3993
Copy link
Collaborator Author

これほとんどそのまま採用しますか...(どこから持ってきた設定何だろう🤔)

一応ある程度オプションの意味軽く調べて、修正・設定適用なりしてPR出そうと思います。PRのときに大事なオプションについては伝えます

@Elise3993
Copy link
Collaborator Author

editorconfig.txt テンプレはこれ使ってました

これですね元
https://xrdnk.hateblo.jp/entry/2020/12/01/191152

@dugite-git
Copy link
Member

これappディレクトリ直下に置く感じですかね

@Elise3993
Copy link
Collaborator Author

Elise3993 commented Jul 25, 2024

外部パッケージが入らない場所の最上階に置きたいからAssets直下になるのかな?

@dugite-git
Copy link
Member

あーたしかに

@Elise3993
Copy link
Collaborator Author

詳細調べたら割と理解に時間かかりそうなので少々お待ちください

https://learn.microsoft.com/ja-jp/dotnet/fundamentals/code-analysis/style-rules/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants