Add pre-merge checklist in core developer guide
メンテナーはふだん 2 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 1/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 62/100
- issue の種類
- ドキュメント
- 明瞭さ
- 明確に書かれている
- 活発さ
- 停滞
調査の方向性
issueにリンクされているdocs/developers/coredev/core_dev_guide.mdのレビューのセクションを編集してください。まずそのセクションを読み、提案するマージ前チェックリストを既存のガイダンスに沿わせます。最終チェックの手順と、その自動化における役割を説明するかどうかを決めてください。ガイドに列挙されたチェックと、最終チェックとマージ待機期間の違いが記載されていれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Our core developer guide has a section on reviewing, but it is missing essential information about merge-time conventions/etiquette. Especially now that @willingc is advocating for getting rid of the "ready to merge" wait period, I think it's important to add to the guide everything that needs to happen before a PR is merged:
- check the type label type (feature, label, performance, etc)
- check the milestone
- check discussion comments are addressed or an issue is made to keep track of remaining to-dos
- make sure the title still reflects the state/purpose of the PR
- make sure the description reflects the current state/purpose of the PR (it's encouraged to leave the original description underneath under the heading "original description", since it provides the context for the subsequent discussion on the PR itself. However, the description becomes the commit message, so it should reflect the status of the PR at the time of merging.)
Some of those checks should ideally be automated, so we could perhaps rename "ready to merge" to "final checks" and keep it only for the automations, not for the wait period.
- 主要言語
- Makefile
- スター
- 13
- フォーク
- 71
- 平均マージ
- 3日 8時間
- マージ済み PR（30日）
- 31
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
napari/docs のほかの issue
-
Link Checker Report対応中かも @rezamuliawijaya が今日担当しました。 オープンmaintenance task
難易度 1/5 1〜3時間 初心者へのやさしさ 75/100
メンテナーはふだん 2 日以内に返信
-
task
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 2 日以内に返信
-
Improve speed of the docs build対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンtask
難易度 4/5 3〜5日 初心者へのやさしさ 53/100
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 57/100
napari/docs#1186 · リアクション 1 件 ·
メンテナーはふだん 2 日以内に返信
似ている issue
-
bug documentation
難易度 1/5 1〜3時間 初心者へのやさしさ 88/100
apify/apify-docs#3206 ·
メンテナーはふだん 1 日以内に返信
-
docs size:S
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信
-
documentation good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
zerohourzulu/irontrellis#16 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 65/100
Droid-Deck/DroidDeck#2198 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
better-auth/better-auth#12292 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信