aboutsummaryrefslogtreecommitdiff
path: root/files/ja/mozilla/developer_guide/how_to_submit_a_patch
diff options
context:
space:
mode:
authorPeter Bengtsson <mail@peterbe.com>2021-04-29 11:14:49 -0400
committerGitHub <noreply@github.com>2021-04-29 16:14:49 +0100
commit50c10e22a2a094f9d46edd56eb64d12f7652246f (patch)
treebd5e7be4288b36a314eb38039261fe973c91468c /files/ja/mozilla/developer_guide/how_to_submit_a_patch
parent7ce9d46289c8931c157dd1f676d427bee517dca2 (diff)
downloadtranslated-content-50c10e22a2a094f9d46edd56eb64d12f7652246f.tar.gz
translated-content-50c10e22a2a094f9d46edd56eb64d12f7652246f.tar.bz2
translated-content-50c10e22a2a094f9d46edd56eb64d12f7652246f.zip
Remove Mozilla/Developer_guide (#691)
Diffstat (limited to 'files/ja/mozilla/developer_guide/how_to_submit_a_patch')
-rw-r--r--files/ja/mozilla/developer_guide/how_to_submit_a_patch/index.html109
1 files changed, 0 insertions, 109 deletions
diff --git a/files/ja/mozilla/developer_guide/how_to_submit_a_patch/index.html b/files/ja/mozilla/developer_guide/how_to_submit_a_patch/index.html
deleted file mode 100644
index 03ed7214bd..0000000000
--- a/files/ja/mozilla/developer_guide/how_to_submit_a_patch/index.html
+++ /dev/null
@@ -1,109 +0,0 @@
----
-title: How to Submit a Patch
-slug: Mozilla/Developer_Guide/How_to_Submit_a_Patch
-tags:
- - Developing Mozilla
- - MDC Project
- - NeedsContent
- - 要更新
----
-<div class="summary">
-<p><span class="seoSummary">パッチを提出し、レビューを受け、Mozilla ソースツリーにコミットして貢献するための幾つかのステップです。この記事ではそのやり方を説明します。</span></p>
-</div>
-
-<p>パッチの提出プロセスは以下の図に書いてある通りです。そして、各ステップの詳細は次のセクションで説明します。</p>
-
-<p style="text-align: center;"><img alt="Workflow of submitting a patch: Preparation | Module Ownership | Creating a patch
-| Testing | Getting Reviews | Addressing Review Comments | Committing the Patch" class="default internal" src="/@api/deki/files/3599/=submitting-patch-workflow.png" style="height: 348px; width: 267px;"></p>
-
-<h2 id="準備">準備</h2>
-
-<p>全てのコードの変更は <a class="link-https" href="https://bugzilla.mozilla.org/" title="https://bugzilla.mozilla.org/">bugzilla.mozilla.org</a> のバグによってトラッキングされています。バグがないものについてはコードレビューされず、レビューされないコードは受け付けません。重複を避けるために変更しようとしている内容が既に存在していないか、<a href="https://bugzilla.mozilla.org/query.cgi?format=specific">バグの検索</a>をし存在しなければファイル(登録)してください。変更についてのほとんどの議論はバグ上で行います。そのため、バグには問題を正しく記載することが解決の鍵です。</p>
-
-<p>バグが正しいプロダクト・コンポーネントとして登録されているか確認してください。もっと情報が欲しい時は、IRC チャンネルの #developers か、ニュースグループに問い合わせてください。</p>
-
-<p>バグを作業している人は bugzilla 内のバグの "assignee" として登録されます。もし誰かがバグにアサインされていれば、その人に変更を調整するためにメールしてください。もしいなければ、バグの状態をあなたが作業していることがわかるようにステータスを変更し、バグの編集権限がある人に自分がアサインでいるよう依頼してください。</p>
-
-<p>パッチが長いこと作られず他のコントリビュータが手をつけないことを避けるため、幾つかのチームでは、アサインされる前にパッチを添付する新しいコントリビュータを待っています。バグのコメントに興味を持つようなことを書くと、チームの誰かが普段使っているプロセスを教えてくれるかもしれません。</p>
-
-<h2 id="モジュールオーナーシップ">モジュールオーナーシップ</h2>
-
-<p>全てのコードは<a href="/ja/docs/">モジュールオーナー</a>によって管理されています。モジュールオーナーはレビューをして変更を許可する役割を持っています。コードを書く前に、モジュールオーナーを確認し、変更提案を許可するか書くにしてください。彼らは新しいユーザーインターフェイス(UI review)、関数(API review)、または変更のテストケースを作成する人を探しています。</p>
-
-<p>もしモジュールオーナーがわからない場合、IRC または ニューズグループに確認した方が良いです。変更するファイル履歴も活用できます。例えば、{{ Source("browser/base/content/browser.js") }} の変更ログを <a href="http://mxr.mozilla.org/mozilla/source/">MXR</a> の "Hg Log" リンクをクリックまたは、<code> hg log browser/base/content/browser.js </code>を実行することで確認します。"r=nickname" のようなログがチェックインメッセージに含まれていてそこからレビューアが誰かを質問することができます。</p>
-
-<h2 id="パッチを作成する">パッチを作成する</h2>
-
-<p>Mozilla ソースコードの変更はパッチで表現されます。パッチはバージョンコントロールにコミットするための基本です。</p>
-
-<p>各パッチは1つの完全な変更を表現し、分割された変更だったらパッチも複数に分割します。もし変更が大きい場合、パッチは複雑になります。その時は、<a href="https://secure.phabricator.com/book/phabflavor/article/writing_reviewable_code/#many-small-commits">パッチを分割し、小さく簡単に読みやすいパッチに変更するためのステップ</a>を参考にしてください。これは変更のレビューを<a href="http://groups.google.com/group/mozilla.dev.planning/msg/2f99460f57f776ef?hl=en">簡単にし、早いレビューを導き</a>、公式なレビューの良い方法です。</p>
-
-<div class="note">
-<p><strong>Note</strong>: 適切なフォーマットでレビューを簡素化させるためのパッチを作るための情報については、<a href="/docs/Mercurial_FAQ#How_can_I_generate_a_patch_for_somebody_else_to_check-in_for_me.3F">patch の精製方法</a>を参照してください。我々の<a href="/docs/Developer_Guide/Reviewer_Checklist">レビューアチェックリスト</a>も参照して良いパッチの作成情報を見てください。</p>
-</div>
-
-<h2 id="テスト">テスト</h2>
-
-<p>すべての変更をテストします。多くの場合、他の変更時にテストできるように<a href="/en-US/docs/Mozilla_automated_testing">自動テスト</a>が要求されます。自動テストができない場合、<a href="/docs/Litmus_tests">Limus tests</a> と呼ばれる手動テストを使ってください。</p>
-
-<p>変更がレグレッションを起こさないようにローカルで自動テストをするか、<a href="https://wiki.mozilla.org/Build:TryServer">Mozilla の try server</a> を使ってください。try server の権限がない場合でも、モジュールオーナーまたは開発者が IRC でジョブを動かすための手助けをしてくれます。</p>
-
-<h2 id="パッチを提出する">パッチを提出する</h2>
-
-<p>もしコントリビュータなら、パッチ提出に MozReview を使うべきです。使い方は <a href="https://mozilla-version-control-tools.readthedocs.org/en/latest/mozreview-user.html">MozReview User Guide</a> を見てください。</p>
-
-<p>もし Mercurial を使う場合、Bugzilla にパッチを添付する仕組みである <a href="https://mozilla-version-control-tools.readthedocs.org/en/latest/hgext.html#bzexport">bzexport extension</a> をインストールします。bzexport の簡単なインストール方法は <code>mach mercurial-setup </code>を実行することです。その後、ターミナルから画面を切り替えることなく、Bugzilla にパッチをアップロードするために <code>hg bzexport -e</code> を実行します。</p>
-
-<p>もし、MozReview や bzexport を使わない場合、bugzilla の <em>Add an attachment </em>リンクを使ってバグにパッチを添付します。レビューアや bugzilla ユーザーがパッチを読めるように <em>patch</em> チェックボックスにチェックを入れてください。</p>
-
-<p>一部のパッチのアプローチ手段を確認したり、フィードバックを問い合わせすることを恐れないでください。コードと一緒に質問をされた時にコメントしたり提案することはとても簡単なことです。</p>
-
-<h2 id="Getting_the_patch_reviewed" name="Getting_the_patch_reviewed">レビューを受ける</h2>
-
-<p>コードレビューを通すことは Mozilla の品質管理の一つです。すべてのパッチはモジュールオーナーまたは、支持したピアによってレビューされます。モジュールを跨いだり、API を変更したり、セキュリティ関係の変更をする場合、レビューに加え、<a class="external" href="http://www.mozilla.org/hacking/reviewers.html">super-review</a> も必須です。</p>
-
-<p>レビュープロセスについての詳しい情報は、<a class="internal" href="/en-US/docs/Code_Review_FAQ" title="/en-US/docs/Code Review FAQ">code review FAQ</a> をご覧ください。もし変更がユーザーインターフェイスに影響を与える場合、<a href="/docs/Developer_Guide/Requesting_feedback_and_ui-review_for_desktop_Firefox_front-end_changes">フィードバッグやデスクトップ版 Firefox のフロントエンド変更のためによる ui-review を要求してください</a>。<br>
- <br>
- <br>
- <img alt="request-review.png" class="internal rwrap" src="/@api/deki/files/3598/=request-review.png" style="border: 1px solid black; float: right; height: 149px; width: 519px;">レビューやスーパーレビューを要求するには、bugzilla の添付ファイルリンクの詳細(Details)をクリックします。左側にラベルと共にドロップダウンが存在します。"review" を見つけ、フラグを"?"に変更し、モジュールオーナーかパッチをレビューしてくれるピアのメールアドレスを入力します。最後に送信することを忘れないでください!</p>
-
-<p><em>注意:</em> もしレビューアが1週間ほど応答しない場合は以下の方法をとってください:</p>
-
-<ul>
- <li>Moizlla の IRC サーバーの <a class="link-irc" href="irc://irc.mozilla.org/developers">#developers</a> に参加し、誰かレビューが遅れている理由を知っているか聞いてください。(その時に一緒にバグのリンクを添えてください)</li>
- <li>もしレビューがされていなければ、直接レビューアにメールをして、いつレビューができるか、または誰か他にレビューができないか聞いてください。</li>
- <li>最後の手段としてnews.mozilla.org のニュースグループに問い合わせてみてください。</li>
-</ul>
-
-<h2 id="レビューコメントを修正する">レビューコメントを修正する</h2>
-
-<p>最初っからパッチがパーフェクトな事はありません。レビューアは "review-" をつけ、そしてパッチを許可する前に修正すべき問題点を教えてくれます。要求しているリビジョンは参加を阻害する意図ではなく、変更の完成度を可能な限り良いものにするために奨励しているということを忘れないでください。レビューアの助言を注意深く修正し、新しいパッチを添付して再度レビューを出してください。</p>
-
-<p>時々レビューアは"review+" と共に、スペルミスやインデントミスなどの修正すべき小さな変更を指摘する場合があります。全ての指摘を修正するべきですが、再レビューは不要です。変更を作成し、リビジョンを改訂したのちに、添付ファイルページに表示されている元のパッチの obsolete チェックボックスにチェックを入れてください。もしそのリビジョンに不安がある場合は、レビューを要求してください。</p>
-
-<p>時々レビューした後で、コミット前の場合、誰かが衝突する変更を加えることがあります。もしマージがシンプルで影響を与えないような場合、アップデートしたバージョンのパッチを投稿します。それ以外はレビューが必要です。</p>
-
-<p>もしレビュープロセスが2週間以上止まっている場合は、上述している "注意" を参考にしてください。</p>
-
-<p>多くのオープンソースプロジェクトで、開発者は完全な状態でもパッチを許可し、それを完了し適用ます。Mozilla の文化では、<strong>レビューアはレビューだけして、パッチにコメント</strong>します。もし パッチ提出者が作成したリビジョンを諦めた場合、誰かがバグを引き取るまではパッチは存在し続けます。</p>
-
-<h2 id="Getting_the_patch_checked_into_the_tree" name="Getting_the_patch_checked_into_the_tree">パッチをコミットする</h2>
-
-<p>レビューが完了したらパッチはコミットできます。</p>
-
-<div class="note">
-<p>パッチが適用されたアプリケーションをビルドしてください。それが動作させて自動テスト(そして完了したバグの動作)を確認し、 <a class="link-https" href="https://wiki.mozilla.org/Build:TryServerAsBranch" title="https://wiki.mozilla.org/Build:TryServerAsBranch">try server</a> にプッシュしてください。(この時もコメントにバグを明記します)<br>
- テストしないパッチを提出するとコミッターの時間を浪費し、ツリーを炎上させます。全ての必要な確認を完了することで、全員の時間と努力を節約してください。</p>
-
-<p>コミッターがパッチをチェックしやすいように、パッチを<a href="/docs/Creating_a_patch_that_can_be_checked_in">適切なフォーマット</a>で作成してください。</p>
-</div>
-
-<p>バグに <a class="link-https" href="https://bugzilla.mozilla.org/describekeywords.cgi#checkin-needed"><code>checkin-needed</code></a> キーワードをつけます。(またはバグの変更権限がない場合は、誰かに追加してもらえるよう訪ねてください) コミット権限を持ったコミュニティーメンバーは通常1日程度で checkin-needed キーワードを持ったバグを見つけ、コミットします。もしパッチが時間帯によってチェックインされていない場合は、<a class="external" href="http://irc.mozilla.org/">irc.mozilla.org</a> の <a class="external" href="http://irc.mozilla.org/developers">#developers</a> に参加し、あなたの代わりにチェックインできる人を聞いてください。ほとんどの場合、ランド後に新しい問題をパッチが引き起こさないことを確認するために Try の実行結果のリンクが必要です。</p>
-
-<p>もし自身でコミットできる場合は、<a href="/en-US/docs/Developer_Guide/Committing_Rules_and_Responsibilities" title="/en-US/docs/Developer Guide/Committing Rules and Responsibilities">Committing Rules and Responsibilities</a> に従うことを忘れないでください。</p>
-
-<h2 id="Getting_the_patch_checked_into_the_tree" name="Getting_the_patch_checked_into_the_tree">レグレッション</h2>
-
-<p>自身の変更により、機能またはパフォーマンスのレグレッションが発生した状態です。特にパフォーマンスレグレッションの厳しい<a href="http://www.mozilla.org/hacking/regression-policy.html">ポリシー</a>があります。これは変更したコードがよくバックアウトされてしまうことがあり、修正して再提出する必要があるということです。レグレッションはテスト(自身がチェックイン前に行ったもの、また実施していないもの)が包括的に不十分ということを意味し、パッチを再提出するか、適切なテストを伴うレグレッションの修正をする必要があります。</p>
-
-<p>いくつかパッチを作成したら、<a href="http://www.mozilla.org/hacking/committer/">Mozilla source code のコミット権を取得する</a>ことも考えてみてください。</p>