2016-04-25 11 views
0

私はコードレビューのニーズにgithubプルリクエストを使用しています。私の唯一の問題は、人がインデント用にタブまたはスペースを使用しているかどうかを知ることができないことです。ここには標準があり、誤ったコードを使用した場合のコードレビューに失敗する可能性があります。彼らはgithubでどちらを使用しているかを知る方法はありますか、またはエディタでファイルを手動で開いてその違いを伝える必要がありますか?プルリクエストコードレビュー時の文字間隔

+0

トラビスを使用して、そこに正しい空白がないかチェックしてください。これは手動ではなく自動的にチェックすることができます。 –

+0

私は通常、この種のスタイルガイドの遵守(jslint、rubocopなど)のために言語固有のリンターを使用しました。これらはTravis – jake

+0

のようなCIツールで実行することができます。少なくとも次の3か4か月の選択肢ではありません。これが、あなたがgithubでこれを行うことができるかどうか尋ねている理由です。混乱させて申し訳ありません。 –

答えて

1

githubでどのファイルを使用しているかを知る方法はありますか、またはエディタでそのファイルを手動で開き、違いを伝える必要がありますか?

理想的に - どちらもありません。

物事が自動化された方法で確認できるときはいつでも、コンピュータに任せてください。多くの他の静的ルール間の空白の適切な使用をチェックすることは、しばしばリンターと呼ばれる様々なツールでチェックすることができます。これはプロジェクトがどの言語を使用しているかによって大きく異なります。もちろんあなたが選んだ場合は、独自のスクリプトを書くこともできます。

Githubでできることは、TravisなどのCIツールにリポジトリを接続することです。これにより、自動的にすべてのプルリクエストを構築し、空白ルールなどのチェックを行うことができます。また、テストスイートやコードの書式設定などを実行することもできます。自動化できるものであれば、手作業を最小限に抑えることができます。

+0

私はこの答えに完全に同意します。もう一年ぐらい、これは間違いないでしょう。しかし、私の会社での管理ができない状況のため、これは少なくとも次の3か4か月間の選択肢ではありません。これが、あなたがgithubでこれを行うことができるかどうか尋ねている理由です。混乱させて申し訳ありません。 –

+0

Travisが現在オプションではない場合は、独自のGithub統合を作成して、これらのことをチェックするスクリプトを実行することができます。しかし、Github自体のエディタを実際に変更する限り、AFAIKは不可能です。必要なAPIについては、[here](https://developer.github.com/v3/repos/hooks/)を参照してください。 –

+0

Githubページ自体に絶対必要で、手動によるチェックがうまくいけば、間違った空白を強調する簡単なブラウザスクリプト/プラグインまたはJavascriptプラグインを書くことができます。 –