質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.75%

  • PHP

    19178questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

  • JavaScript

    15275questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • Java

    13145questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • Ruby

    7017questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Node.js

    1731questions

    Node.jsとはGoogleのV8 JavaScriptエンジンを使用しているサーバーサイドのイベント駆動型プログラムです。

コード整形コマンドの実行はgitログに悪影響ですか?

受付中

回答 4

投稿

  • 評価
  • クリップ 0
  • VIEW 398
退会済みユーザー

退会済みユーザー

javascriptでprettierというコードを整形してくれるライブラリをつかってます。これを実行すると対象ディレクトリのコード全体が綺麗にフォーマットされるのですが、gitを他の人と共有して使う際に迷惑になりますか?

どうすればよいでしょう。

マスターブランチにマージするときだけ、コードの整形を走らせるとかにすれば良いのでしょうか?

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 4

+10

あなたのプロジェクトの中に「オレオレフォーマット」を書いている人が居ると、その人に迷惑がかかるでしょう。

といっても、そういった「一人のわがまま」を取るより「その他全員の幸福」を取るべきだと私は思います。「すべてのコードが統一のフォーマットで書かれている」というのはとてもメリットのあることで、コードを追う際の効率に関わります。ですので、全員が共通のコード整形ツールを使うことを呼びかけるのが一番よいです。保存時に自動でコード整形ツールが走るようになっていれば整形漏れもなくなるのでなおよいです。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/10/19 19:03

    gitのログがおいづらくなるのが心配で

    キャンセル

  • 2017/10/20 00:39

    確かにフォーマットをかける前、後で追いづらくなるのは事実です。しかし、きちんと順を追っていけば追えないこともないです。
    私の経験談ですが、はっきり言ってログをそこまで遡って追うことはほぼ発生しないですし、そのためにフォーマットをしないことの方がよほど作業効率に支障が出ます。一度フォーマットをかけて、以後フォーマットを自動でかけることを徹底すればそこからのログはきれいなものです。なので、ちゃんとみんなが一斉にフォーマットを導入すれば、ログを追いづらくなる壁は一度しか発生しません。

    キャンセル

+4

もし以前からあるコードに対して整形を掛けていくのであれば、コンフリクトの解消やコミットログの連続性の消失という痛みが伴います
ほかにも、そのツールの導入・共通化の時間や労力が必要になるでしょう(一人が使うだけでは効果的ではありません)

そこで、たとえば GitHub + SideCI の組み合わせが考えられるかもしれません

「コードの整形ができていない場合、それが出来るまでこのブランチはマージできません」といった判定を新たに設けることができます
(書いたコードに対するテストコードの通過失敗と同じくらいの意味合い)

このワークフローや類似のものを確立できれば、各編集者にその作業が必要であることを機械的に通知することができます

コンフリクトの解消を低頻度長期的に実施しつつ、新たに書かれるコードはCI によるチェックが入った整形済みのものがコミットされる、そういった環境にしていけると思います

何か参考になれば幸いです

 Links

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

+3

整形による編集と、整形前のコードに対して行われた編集が高い確率でコンフリクトしますので、ローカルのリポジトリにあるまだpushしていない編集と、別ブランチから/へのマージがほぼもれなくコンフリクトします。

コードを共有している場合、関係者全員が困ることになります。

また、たいていの場合ファイル全体にわたって編集が行われるため、あるコードが以前に編集されたのはいつか、という情報がほぼリセットされます。したがってコードの連続性を追跡するのが難しくなります。

これはコードを共有していなくても発生します。

--
基本的には既に存在するコードベースに整形ツールを掛けるのはなかなか難しいです。ローカルの修正はpush/pullしてもらえばいいとして(それも難しい、という状況もあり得ますが)、ブランチが問題になります。

例えばmasterと以前のリリースのメンテナンスブランチがあるような時に、どちらか一方だけ整形すると、どちらにも必要な修正を行うのに苦労します。両方整形すると、メンテナンスブランチに大規模な変更を突っ込んでしまうことになります。

現実的には

  • デメリットは甘受して整形ツール掛けてしまう
  • 既存コードへの全面適用は諦め、新規のコードや変更箇所のみ整形ツールを掛けるようにする

を状況によって判断することになるかとおもいます。

履歴の断絶について、例えば「ある部分に見つかったバグはどのcommitによるものか」という調査がログからはできないことになります。リポジトリが存在する限り永遠につきまとうことになります。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/10/19 19:03

    やっぱそうですよね。整形を実行して良いタイミングってチーム開発において何かありますか?何かのタイミングにhookさせるとか。

    キャンセル

+3

javascriptでprettierというコードを整形してくれるライブラリをつかってます。

プロジェクトの決定は法律と同じで、悪法もまた法なり
これは個人の思想がどんな優れていても同じ

もしこれがプロジェクトの決定ならば、
「ツールで整形せずにコミットした奴は説教」レベルの命令違反だね。

gitを他の人と共有して使う際に迷惑になりますか?

でもこの1文がなんか怖いんだよねぇ…
もしprettierを使おうとしているのがhayatomoさんの独断なら
「お前何勝手なことしてコンフリクト作りまくってるんだ」と叱られる側になっちゃうよ!

使うなら使うで皆に合意を取ってプロジェクトの決定にするべき。
ツールで整形すること自体は良い事だから、頑張ってプロジェクトの決定まで持っていって欲しい。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/10/22 15:17

    > ツールで整形すること自体は良い事だから、頑張ってプロジェクトの決定まで持っていって欲しい。
    そうしてみます!!!ありがとうございました!

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.75%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • PHP

    19178questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

  • JavaScript

    15275questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • Java

    13145questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • Ruby

    7017questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Node.js

    1731questions

    Node.jsとはGoogleのV8 JavaScriptエンジンを使用しているサーバーサイドのイベント駆動型プログラムです。