javascriptでprettierというコードを整形してくれるライブラリをつかってます。これを実行すると対象ディレクトリのコード全体が綺麗にフォーマットされるのですが、gitを他の人と共有して使う際に迷惑になりますか?
どうすればよいでしょう。
マスターブランチにマージするときだけ、コードの整形を走らせるとかにすれば良いのでしょうか?
気になる質問をクリップする
クリップした質問は、後からいつでもMYページで確認できます。
またクリップした質問に回答があった際、通知やメールを受け取ることができます。
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
回答4件
0
あなたのプロジェクトの中に「オレオレフォーマット」を書いている人が居ると、その人に迷惑がかかるでしょう。
といっても、そういった「一人のわがまま」を取るより「その他全員の幸福」を取るべきだと私は思います。「すべてのコードが統一のフォーマットで書かれている」というのはとてもメリットのあることで、コードを追う際の効率に関わります。ですので、全員が共通のコード整形ツールを使うことを呼びかけるのが一番よいです。保存時に自動でコード整形ツールが走るようになっていれば整形漏れもなくなるのでなおよいです。
投稿2017/10/19 05:34
編集2017/10/19 05:36総合スコア9206
0
もし以前からあるコードに対して整形を掛けていくのであれば、コンフリクトの解消やコミットログの連続性の消失という痛みが伴います
ほかにも、そのツールの導入・共通化の時間や労力が必要になるでしょう(一人が使うだけでは効果的ではありません)
そこで、たとえば GitHub + SideCI の組み合わせが考えられるかもしれません
「コードの整形ができていない場合、それが出来るまでこのブランチはマージできません」といった判定を新たに設けることができます
(書いたコードに対するテストコードの通過失敗と同じくらいの意味合い)
このワークフローや類似のものを確立できれば、各編集者にその作業が必要であることを機械的に通知することができます
コンフリクトの解消を低頻度長期的に実施しつつ、新たに書かれるコードはCI によるチェックが入った整形済みのものがコミットされる、そういった環境にしていけると思います
何か参考になれば幸いです
Links
投稿2017/10/19 14:32
総合スコア2321
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
0
javascriptでprettierというコードを整形してくれるライブラリをつかってます。
プロジェクトの決定は法律と同じで、悪法もまた法なり
これは個人の思想がどんな優れていても同じ
もしこれがプロジェクトの決定ならば、
「ツールで整形せずにコミットした奴は説教」レベルの命令違反だね。
gitを他の人と共有して使う際に迷惑になりますか?
でもこの1文がなんか怖いんだよねぇ…
もしprettierを使おうとしているのがhayatomoさんの独断なら
「お前何勝手なことしてコンフリクト作りまくってるんだ」と叱られる側になっちゃうよ!
使うなら使うで皆に合意を取ってプロジェクトの決定にするべき。
ツールで整形すること自体は良い事だから、頑張ってプロジェクトの決定まで持っていって欲しい。
投稿2017/10/19 15:06
総合スコア21158
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
0
整形による編集と、整形前のコードに対して行われた編集が高い確率でコンフリクトしますので、ローカルのリポジトリにあるまだpushしていない編集と、別ブランチから/へのマージがほぼもれなくコンフリクトします。
コードを共有している場合、関係者全員が困ることになります。
また、たいていの場合ファイル全体にわたって編集が行われるため、あるコードが以前に編集されたのはいつか、という情報がほぼリセットされます。したがってコードの連続性を追跡するのが難しくなります。
これはコードを共有していなくても発生します。
--
基本的には既に存在するコードベースに整形ツールを掛けるのはなかなか難しいです。ローカルの修正はpush/pullしてもらえばいいとして(それも難しい、という状況もあり得ますが)、ブランチが問題になります。
例えばmasterと以前のリリースのメンテナンスブランチがあるような時に、どちらか一方だけ整形すると、どちらにも必要な修正を行うのに苦労します。両方整形すると、メンテナンスブランチに大規模な変更を突っ込んでしまうことになります。
現実的には
- デメリットは甘受して整形ツール掛けてしまう
- 既存コードへの全面適用は諦め、新規のコードや変更箇所のみ整形ツールを掛けるようにする
を状況によって判断することになるかとおもいます。
履歴の断絶について、例えば「ある部分に見つかったバグはどのcommitによるものか」という調査がログからはできないことになります。リポジトリが存在する限り永遠につきまとうことになります。
投稿2017/10/19 07:19
編集2017/10/21 02:50総合スコア1449
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
退会済みユーザー
2017/10/19 10:03
あなたの回答
tips
太字
斜体
打ち消し線
見出し
引用テキストの挿入
コードの挿入
リンクの挿入
リストの挿入
番号リストの挿入
表の挿入
水平線の挿入
プレビュー
質問の解決につながる回答をしましょう。 サンプルコードなど、より具体的な説明があると質問者の理解の助けになります。 また、読む側のことを考えた、分かりやすい文章を心がけましょう。
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
退会済みユーザー
2017/10/19 10:03
2017/10/19 15:39