Document and enforce pull request review guidelines

Aperta Adatta ai principianti
  • Qualcosa non va?
#1,868 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

@dashorst ci sta già lavorando.

Dal 8/11/2026.

  • #1869 di @dashorst — aperta

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
76/100
Tipo di issue
Documentazione
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
java
Ambito
developer-experience, documentation

Direzione di ricerca

Inizia da AGENTS.md nella root del repository e individua come sono organizzate le indicazioni esistenti per contribuire. Prepara un REVIEW.md nella root che copra le sette aree di revisione elencate nell’issue e aggiungi un riferimento al file in AGENTS.md. Il lavoro è completato quando entrambi i file forniscono una checklist di revisione coerente e utilizzabile per i controlli proposti.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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, NOTICE and licenses/ for third-party code or assets copied into the tree.
  • OSGi and JPMS: the bundle manifest and module-info.java stay 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 LinkedMap in MarkupContainer and 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.

Lingua principale
Java
Stelle
799
Fork
420
Merge medio
18h 37m
PR unite (30g)
140

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di apache/wicket

Tutte le issue di apache/wicket

Issue simili

Altre issue su Java

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.