承認者およびメンテナーのための SIG のプラクティス
このページでは、承認者およびメンテナーが使用するガイドラインと一般的なプラクティスについて説明します。
オンボーディング
コントリビューターがドキュメントに対する責任の大きい役割(承認者、メンテナー)を引き受ける場合、既存の承認者およびメンテナーによってオンボーディングが行われます。
docs-approvers(またはdocs-maintainers)グループに追加されます#otel-comms、#otel-maintainers、およびチーム内のプライベート Slack チャンネルに追加されます- SIG Comm ミーティングとメンテナーミーティングのカレンダー招待への登録を求められます
- SIG Comm の現在のミーティング時間が適しているかを確認し、適さない場合は既存の承認者およびメンテナーと調整して、全員に適した時間を決定します
- コントリビューター向けのさまざまなリソースを確認します
- コミュニティリソース。 特に、コミュニティメンバーシップ と ソーシャルメディアガイド に関するドキュメント
- コントリビューションガイドライン。この一環として、これらのドキュメントをレビューし、イシューやプルリクエストを通じて改善するためのフィードバックを提供します
以下は、追加で確認すると有益なリソースです。
- Hugo ドキュメント
- Docsy ドキュメント
- マーケティングガイドライン。これは、Linux Foundation のブランドおよび 商標使用ガイドライン 含みます。 これらは、レジストリ、インテグレーション、ベンダー、導入事例、ディストリビューションのエントリーをレビューする際に特に重要です
コラボレーション
- 承認者およびメンテナーは、それぞれ異なる勤務時間や状況です。 そのため、すべてのコミュニケーションは非同期とみなされ、通常のスケジュール外で返信する義務はありません
- 承認者またはメンテナーが長期間(数日または1週間以上)不在となる場合は、 #otel-comms チャンネルでの通知や GitHub ステータスの更新を行います。
- 承認者およびメンテナーは、OTel 行動規範 とコミュニティの価値観に従います。 彼らはコントリビューターに対して親切かつ協力的であり、何らかの対立や誤解、不快になるような状況が承認者またはメンテナーに生じた場合は、会話、イシューまたは PR から一歩引いて別の承認者やメンテナーに対応を依頼できます
トリアージ
イシュー
- 新規のイシューは
@open-telemetry/docs-triagersチームによってトリアージされます。 - 最初のステップとして、トリアージをする人はイシューのタイトルと説明を読み、以下のラベルを適用します
- 必須:
sig:*,lang:*またはdocs:*ラベルを適用してイシューの所有権を決定します- イシューが SIG によって共同所有されているコンテンツまたは質問と関連する場合は、
sig:*ラベルが適用されます。たとえば、コレクターに関する質問はsig:collectorラベルが適用されます。 - イシューが特定のローカリゼーションに関するコンテンツまたは質問と関連する場合は、
lang:*ラベルが適用されます。 - イシューがドキュメントチーム (SIG Comms) によって単独で所有されているコンテンツまたは質問と関連する場合は、
docs:*ラベルが適用されますdocsdocs:admindocs:accessibilitydocs:analytics-and-seodocs:IAdocs:blogdocs:cleanup/refactoringdocs:upstream,docs:upstream/docsydocs:javascriptdocs:mobiledocs:registrydocs:ux
- イシューが SIG によって共同所有されているコンテンツまたは質問と関連する場合は、
- 必須:
triage:*ラベルtriage:accepted、triage:accepted:needs-prtriage:deciding、triage:deciding:blocked、triage:deciding:needs-infotriage:rejected、triage:rejected:duplicate、triage:rejected:invalid、triage:rejected:wontfix
- 必須: イシューの “type” を以下のように設定します
- バグには
bugイシュータイプ - 機能要望には
enhancementイシュータイプ - 質問には
type:questionラベル - 文章校正には
type:copyeditラベル - 作業として対応できない、方向性の定まらない議論の場合は、イシューを “discussions” に移動します
- バグには
- 任意: 該当する場合は、以下の見積もりラベルを適用します
e0-minutes- …
e4-months
- 任意 (メンテナーのみ): 優先度ラベル
p0-criticalp1-highp2-mediump3-low
- 任意: 以下の特殊タグのいずれか
good first issuehelp wantedcontribfestmaintainers onlyforeverstale
- 必須:
- イシュー上で 14 日間活動がない場合、自動化により
triage:deciding状態のイシューにtriage:followupラベルが付与され、再トリアージの対象となります。triage:followupラベルは 7 日以内に削除する必要があります。 参加者へのメンションとラベルの削除で、十分な活動とみなされます。
PR
- 以下の例外を除いて、PR は
triage:acceptedラベルが付与されたイシューとリンクされている必要があります- 自動的な PR
- メンテナーまたは承認者によるホットフィックス
- 自動化により、PR は適切な共同所有 SIG またはローカリゼーションチームにラベル付けおよび割り当てされます。
- PR はイシューと同じ共同所有ラベルをもつ必要があります
- SIG により PR が共同所有されている場合、このグループは、内容が技術的に正しいかを確認するための最初のレビューを行う責任があります。
- 言語チームにより PR が共同所有されている場合、このグループは、内容の翻訳が正確かどうかを確認するための最初のレビューを行う責任があります。
- ドキュメントチームの主な責務は、PR がプロジェクトの全体目標に沿っているか、構造内の適切な場所に配置されているか、スタイルおよびコンテンツガイドに従っているかを確認することです。
- マージに必要なものが欠けている PR には、適切なラベルを付与する必要があります
missing:clamissing:docs-approvalmissing:sig-approvalblocked
- PR 上で 21 日間活動がない場合、自動化により
staleラベルが付与され、再レビューが求められます。staleラベルは 14 日以内に削除する必要があります。 参加者へのメンションとラベルの削除で、十分な活動とみなされます。 - PR は決して自動的に閉じられません。
コードレビュー
一般
PR のブランチが
out-of-date with the base branch(ベースブランチと同期していない)場合、継続的に更新する必要はありません。 各更新ごとに CI チェックが再実行されます! マージ前に更新すれば十分です非メンテナーによる PR で git サブモジュールを決して更新しないでください。 これは時々誤って起こります。 PR 作成者にその必要はないことを伝えてください。 マージ前にこれを修正しますが、将来的には最新のフォークから作業する必要があります
コントリビューターが CLA に署名できない場合、またはコミットの 1 つで誤って間違ったメールアドレスを使用した場合は、問題を修正するか、プルリクエストをリベースするように依頼してください。 最悪の場合、PR を閉じて再度開き、新しい CLA チェックをトリガーします
cSpell に未知の単語は、以下の 3 つのうちいずれかの ignore リストに追加できます
- ページフロントマター: 一般的に、1 度しか現れない単語に適しています。 詳細はスペルチェックを参照してください。
- ロケール辞書:
.cspell/en-words.txtのように、同じ言語における複数のページで使用される単語に適しています。 - 共通 (すべてのロケール) 単語リスト:
.cspell/all-words.txt。 製品名や人名のように、スペルがすべての言語で有効な単語に適しています。 詳細はスペルチェックを参照してください。
レビュワーと承認者は、レビュー中に配置が適切かどうかを判断できます。
共同所有 PR
SIG が共同所有するドキュメント(コレクター、デモ、言語固有など)を変更する PR は、ドキュメント承認者による承認と SIG 承認者による承認の 2 つの承認を目指す必要があります。
- ドキュメント承認者は PR に
sig:<name>ラベルを付与し、SIG-approversグループのタグ付与します - ドキュメント承認者が PR を承認した後、
sig-approval-missingラベルを追加します。 これにより SIG に対応を促します - SIG の承認が一定期間(通常 2 週間、緊急の場合は短縮可能)内に得られない場合、ドキュメントメンテナーの判断でマージできます
ボットによる PR
ボットが作成した PR は以下の方法でマージできます。
- レジストリのバージョンを自動更新する PR は、即座に修正、承認、マージ可能です
- SDK、ゼロコード計装、またはコレクターのバージョンを自動更新する PR は、対応する SIG がマージを延期する必要があることを通知しない限り、承認およびマージできます
- 仕様のバージョンを自動更新する PR では、CI チェックに合格するためにスクリプトの更新が必要になることがよくあります。 その場合は @chalin が対応します。 それ以外の場合は、対応する SIG がマージを延期する必要があることを通知しない限り、それらの PR は承認され、マージされる可能性があります
翻訳 PR
翻訳の変更を含む PR は、2 人の承認を目指します。 1 人はドキュメント承認者で、1 人は翻訳承認者です。 共同所有 PR の手順と同様のプラクティスが適用されます。
PR のマージ
メンテナーは以下のワークフローを適用して PR をマージできます。
PR がすべての承認を得ており、CI チェックが通っていることを確認します
ブランチが最新でない場合、GitHub UI からリベースで更新を行います
更新により CI チェックが再実行されるため、チェックが成功するのを待つか、次のようなスクリプトを使用してバックグラウンドで実行します。
export PR=<PR の ID>; gh pr checks ${PR} --watch && gh pr merge ${PR} --squash
仕様の PR と統合ブランチ
ウェブサイトは、opentelemetry-specification と semantic-conventions リポジトリからの未リリースの変更を連続的に統合しています。 2 つのスケジュールされたワークフロー (詳細) が毎日実行され、ドラフトの “integration” PR を次の dev バージョンに合わせて最新の状態に保ちます。
- ブランチパターン:
otelbot/spec-integration-vX.Y.Z-devおよびotelbot/semconv-integration-vX.Y.Z-dev。 - 稼働中のブランチ一覧: spec · semconv。
Spec / semconv SIG メンテナー
リリース直前に以下の手順を実施します。
- 前のセクションのリンクから、たとえば
otelbot/spec-integration-v1.56.0-devのような、対象となる仕様の最新の統合ブランチをみつけます。 - ブランチページからリンクされている、関連 PR を開きます。
- 対応するワークフローを新たに実行し、最新の変更を取り込みます
- PR チェックが成功している場合、仕様はリリース可能です。
失敗している場合は、
@open-telemetry/docs-maintainersにメンションし、リリース前に不具合を解消できるようにします。
Comms SIG メンテナー
月を通して、統合 PR を定期的に確認し、CI チェックが成功するよう段階的な修正をコミットしてください。 アップストリームの変更セットがまだ小さいうちに、不具合を早期に察知する方が、リリース当日に慌てて対応するよりもはるかに簡単です。
フィードバック
このページは役に立ちましたか?
Thank you. Your feedback is appreciated!
Please let us know how we can improve this page. Your feedback is appreciated!