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

改行コードについて整理する #162

Open
meltingrabbit opened this issue Jan 4, 2022 · 4 comments
Open

改行コードについて整理する #162

meltingrabbit opened this issue Jan 4, 2022 · 4 comments
Assignees
Labels

Comments

@meltingrabbit
Copy link
Collaborator

meltingrabbit commented Jan 4, 2022

概要

コーディング規約に

- 改行コードはCR+LFで統一する.

とあるが,現状これはWindows環境でcloneすると auto crlfがかかってるだけなので,repository上ではLFになっている気がする.

機械的に統一/チェックをする,もしくはコーディング規約を変えるのか,といったことを整理したい.

詳細

  • 現状のコーディング規約ではCR+LFに統一することを要求している
  • この規約は機能/強制しておらず,現状のリモートリポジトリのソースコードのファイルの改行コードはLFになっている
  • CR+LFは特定の開発環境(Renesas)から来ている制約であり,本当に要求されるのはここでビルドする際にCR+LFになっていることのみ
  • Gitの設定でcore.autocrlf=trueになっていれば作業時にはCR+LFになる
  • Git for Windowsのデフォルト設定がcore.autocrlf=trueであるため,なんとかなってはいる
  • 改行コードを何にするにしても,これは各自の設定任せではなく.gitattributesで設定すべき

関連

close条件

整理できたら

@sksat
Copy link
Collaborator

sksat commented Jan 4, 2022

現状もリモートリポジトリ全部LFっぽいですし,規約の方を変えて.gitattributes追加だけでよさそう?

@meltingrabbit
Copy link
Collaborator Author

その説が濃厚だと思ってる.

@meltingrabbit
Copy link
Collaborator Author

これちゃっちゃとやっちゃうか.next core update欄に移動した.

@meltingrabbit meltingrabbit self-assigned this Jan 4, 2022
@meltingrabbit
Copy link
Collaborator Author

適当に関係者と話して,この方向で行くことにした

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

No branches or pull requests

2 participants