2009-05-19 18 views
0

免責事項私はコードレビューをしていないので、この質問は単なる学問的関心事です。コードレビューの前にコードを保存する場所はどこですか?

私は、ソースコントロールに許可されているの前に、コードレビューを提唱し、必要とするstackoverflowの投稿をたくさん見てきました。

これを行うと、レビューされていないコードをどこに保存するのですか?また、他のチェックによるコンフリクトを処理するために開発者がコードを更新する必要がある場合の対処方法コードをもう一度見直した?

ありがとうございます。

答えて

4

コードはあなたのリポジトリ(SVN、TFSなど)にチェックインされます。レビュー後にトランクに入るのを防ぐために、開発(または開発者単位でも)ブランチを設定することができます。

3

レビューされていないコードはまだテストされています。信頼は開発にとって重要なことです。答えは、コードがバージョン管理システムにのみ属しているということです。そこからいくつかのレビューを取得します。問題がある場合、ロールバックすることができます。他のすべては、大部分の時間のための大きな喧騒です。リポジトリ内のコードは「生産中のコード」ではないことを覚えておいてください。

コミットする前に誰もがリポジトリから更新する必要があります。多くの更新を検出した場合は、テストを再度実行することが本当に実現可能です。コミットよりも問題がなければ

-2

開発者のデスクトップに保存します。まだ完了していないので、SVNにはチェックインされていません。

コードが多すぎると問題になります

妥当な量のコードであれば、レビューアに電子メールで送信することができます。たぶん、整理しておくためにZIPファイルが必要かもしれません。

SharePointに投稿することもありますが、それはまれです。電子メールは通常正常に動作します。

+0

コードはまだ実行されていなくても価値があります(たとえば、会社または開発チームの場合)。それがバージョン管理されるまでは、バージョン管理が提供する利点は得られません。 –

+0

@Craig McQueen:我々の立場は、レビューされるまで価値がないということです。最初のレビューはほとんど常にリワークにつながります。いくつかのケースでは、重要な "そのモジュールを破棄し、代わりにこのように行う"という種類のリワークです。コードが実際に価値を持っているかどうかは、単なる政策決定であり、私たちは別の選択をしました。 –

0

私の現在のチームはステークホルダーに結果を届ける前にコードのレビューを行っているので、私はコードレビューの後でのみソース管理を行うことを主張するキャンプにいる。

つまり、ソースファイルにコミットするのではなく、ディスク上のディレクトリにパッチファイルを保存することが考えられます。別のオプションは、変更がターゲットブランチにマージされる前にコミットされている別のブランチを使用することですが、このアプローチが危険にさらされていることを恐れています。

1

コードレビューも行われていないため、学問的質問に対する学問的回答。

EVERYTHINGがソースコントロールにチェックインされます。それが完全に動作していない/テストされている/見直されていない場合、その開発者の個人的な支店に入ります。

1

この種の練習は、GitのようなDVCSを使用する大きな理由です。開発者はコミットせずに長期間作業することができ、コードレビューは可能ですが、われわれが感謝しているバージョン管理技術を使用しています。あなたがSVNのようなものを使用しているなら、すべてのバグ/機能/それが書かれる必要があるものすべてをブランチする必要があり、コードレビューの後に再統合する必要があります...痛いかもしれません。

1

これは実際にチームが設定したツールと手順の種類によって異なります。

非公式コードレビューの場合、バージョンコントロールにチェックインして、他の開発者が自分のスケジュールでそれを確認できるようにするだけです。レビューによって見つけられた問題は別々にチェックされます。

GoogleのRietveldのようなツールを正式に使用するツールについては、スマートベアのCode Collaboratorは、レビュー用のコードをアップロードし、バージョン管理のためのミニチュアバージョンを用意しています(詳しくは、こちらをご覧ください)レビューの過程で更新された各投稿のスタイル履歴を表示します。 (コードレビューの詳細について知りたい場合は、スマートベアにはfree bookがあります)

関連する問題