Come evitare i vecchi commit / PR che sono dolorosi da unire?

Question

Come evitare i vecchi commit / PR che sono dolorosi da unire?

#1 da (1 voti)

1

Abbiamo un buon processo di revisione del codice, ma ci sono un paio di problemi. Il problema più fastidioso è la fusione. Di volta in volta il numero di cose da unire cresce rapidamente e alcuni dei commit stanno invecchiando. Di conseguenza, il rebasing è un dolore (conflitti).

Che cosa possiamo fare per evitarlo? Posso pensare a due possibili soluzioni:

Unione di giovedì. Dalle 10 alle 12 ogni settimana uniamo PR
Avere un timeout su PR. Se è in revisione tra pari per una settimana, invia tutti nella squadra.

git code-reviews scm merging

posta Lukasz Madon 07.01.2015 - 11:24

fonte

1 risposta

Leggi altre domande sui tag git code-reviews scm merging

Applicazioni Web stateless Sconfiggi DBContext in qualche modo? Qual è il modo migliore per creare dinamicamente un DOM?

score 1 · Answer 1

Una soluzione un po 'eccessiva è impedire agli sviluppatori di impegnarsi, mentre devono fare delle recensioni.

Ma come Bart scrive, è davvero un problema organizzativo, dovresti risolverlo ricordando agli sviluppatori che la revisione ha una priorità più alta rispetto alla scrittura di un nuovo codice. Alcuni sviluppatori potrebbero desiderare di avere uno strumento che fornisca un'indicazione "live" di quante recensioni hanno nella propria coda.

Non so come organizzi le recensioni nel tuo gruppo, ma se non assegni recensioni specifiche a sviluppatori specifici, ciò potrebbe portare a tutti coloro che sperano che qualcun altro faccia il lavoro.