Document and enforce pull request review guidelines
メンテナーはふだん 1 日以内に返信
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 76/100
- issue の種類
- ドキュメント
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- java
調査の方向性
リポジトリのルートにある AGENTS.md から始め、既存の貢献ガイドがどのように構成されているかを確認してください。Issue に記載された7つのレビュー領域を網羅するルートの REVIEW.md を作成し、AGENTS.md から参照してください。両方のファイルに、提案されたチェック項目に対する一貫性のある実用的なレビュー用チェックリストが記載されていれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Write down what a review of a Wicket pull request checks, so human and agent reviewers apply the same checks to every change, and make the checklist part of the workflow.
AGENTS.md describes how to write a change: format, commits, build, API compatibility, Javadoc, comments, security. Nothing describes what to check when reviewing one. The checks this issue should cover:
- License compliance: ASF headers, the licenses of new dependencies under the ASF third-party policy, and
LICENSE,LICENSE-binary,NOTICEandlicenses/for third-party code or assets copied into the tree. - OSGi and JPMS: the bundle manifest and
module-info.javastay consistent with the pom when packages or dependencies change. - Performance: a change to a hot path keeps its complexity and memory footprint, or shows with a benchmark that the cost is worth it. During the dependency cleanup for #1862, a first draft replaced
LinkedMapinMarkupContainerand made single child removal O(n). - Accessibility: markup, components and JavaScript that render UI keep meeting WCAG.
- Documentation: changes users notice update the migration guide, user guide or Javadoc in the same pull request.
- Serialization: changes to the serialized form of components and models, which affect stored pages and session size.
- Tests: new test classes end in
Test, or Surefire never runs them.
Proposal: a REVIEW.md in the repository root with these checks, referenced from AGENTS.md so agents writing a change check it against the same list.
- 主要言語
- Java
- スター
- 799
- フォーク
- 420
- 平均マージ
- 18時間 37分
- マージ済み PR（30日）
- 140
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
apache/wicket のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信
-
COOP and COEP exemptions compare with the context path, so page exemptions never match対応中かも @asvanberg が 18 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
apache/wicket#1661 · コメント 1 件 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
-
The CSP report-uri is lost in development mode because strict() clears it再び着手できるかも @VelpulaGangavardhan が 22 日前に担当しましたが、オープン中のプルリクエストはありません。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
apache/wicket#1660 · コメント 5 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信
似ている issue
-
`Time.at(t, in: nil)` returns UTC instead of the local time (or the time zone of `t`), unlike CRubyオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
hermes-queue
難易度 1/5 1時間未満 初心者へのやさしさ 72/100
chrisalunlloyd2-sudo/mindpalace#349 ·
メンテナーはふだん 1 日以内に返信
-
Retry-After parser accepts invalid delay-seconds values対応中かも @Dhruvraj-Rana が今日担当しました。 オープンBug
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
open-telemetry/opentelemetry-java#9018 ·
メンテナーはふだん 2 日以内に返信
-
bug priority: medium Review effort [1-5]: 2 status: needs-triage type: bug
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
carlos-emr/carlos#5818 ·
メンテナーはふだん 1 日以内に返信
-
bug good first issue help wanted tooling
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
appmonster007/kmip-adapter#100 ·