MetaMask / MetaMask/contributor-docs

Add section on stacking pull requests to Pull Requests guide

Aperta
#51 1 commento 1 reazione 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

team-wallet-framework
Lingua principale
JavaScript
Stelle
86
Fork
42
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

If you have a set of changes that you need to split up into separate stages, and some of those stages are dependent on each other, you'll need to create PRs that depend on each other. There's some specific advice around creating stacked pull requests. Specifically,

  • keep all PRs in the stack except the first in draft (because of the next point)
  • you never want to merge a PR that's not the first in the stack into another that's also not the first — when the first PR is merged, you need to rebase all other PRs

Guida per i contributori

Apri la guida per i contributori

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.

Direzione di ricerca

Apri la guida alle Pull Requests e individua la sezione relativa alle pull request dipendenti o impilate. Aggiungi i suggerimenti elencati su come mantenere le PR successive in Draft, evitare i merge tra PR che non siano la prima e fare il rebase dello stack rimanente dopo il merge della prima PR. Il lavoro è completato quando la guida spiega chiaramente queste pratiche.

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

Valutazione

Ambito
documentation
Tipo di issue
Documentazione
Difficoltà
1/5
Tempo stimato
Meno di un'ora
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
50/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.