Presentazione delle richieste pull per un team di 2 persone: unire le mie richieste?


11

Sto presentando git a un membro del team junior (una cooperativa).

Ora sono a loro agio con le basi per aggiungere, impegnare, spingere e tirare.

Ora voglio presentarli per estrarre richieste e rami.

Se iniziano a fare richieste pull nelle filiali, dovrei fare lo stesso per il mio lavoro in corso?
Sarò io a unire le loro richieste pull. Non ero sicuro che avrebbe avuto più senso lavorare nei rami (in genere una buona pratica lo so, ma sono curioso di sapere questa situazione specifica di 2 sviluppatori con uno junior ) e, in tal caso, ciò significa che unirò i miei rami in master. Farei comunque una richiesta pull per il mio lavoro / le mie filiali? Generalmente utilizziamo il flusso di lavoro di base del ramo delle funzioni github per queste modifiche:
https://www.atlassian.com/git/tutorials/comparing-workflows/feature-branch-workflow

C'è uno scopo per usare le richieste pull sul mio repository se sono l'unico sviluppatore? è utile ma non altrettanto specifico.

Qual è il flusso di lavoro con 2 persone su un progetto sembra anche più generale

e

Devo aprire le richieste pull da una filiale sul repository ufficiale o sul mio fork? sembra di più sulle forcelle.

Risposte:


19

No. Non devi unire le tue richieste pull. Ciò che è buono per l'oca è buono per il gander. Unire le tue richieste pull crea un precedente negativo per il nostro sviluppatore junior. Significa anche che nessun altro sta guardando il tuo codice. Non importa quanto siamo anziani, tutti commettiamo errori e scriviamo di tanto in tanto un codice errato. Insegna ai tuoi ragazzi come funzionano le revisioni del codice dall'altra parte facendogli rivedere e unire il tuo lavoro.

Potrebbe non avere lo stesso occhio che hai tu, ma lo abituerà al processo dalla fine del recensore e potrebbe sorprenderti e catturare qualcosa di stupido che hai fatto. In minima parte, ti darà un'indicazione di pezzi di codice che sono ovvi per te, che non sono ovvi per lui. Questo ha un duplice vantaggio.

  1. Entrambi imparate dove il vostro minore ha bisogno di focalizzare le attività di apprendimento.
  2. Impara dove sei più intelligente di quanto dovresti essere.

6
L'altro enorme vantaggio delle revisioni del codice è che almeno due persone hanno visto, conoscono e hanno avuto la possibilità di porre domande su ogni singola modifica del codice prima che avvenga. Anche se lo sviluppatore junior non sa cosa cercare, è garantito per imparare qualcosa da tutto ciò.
Ixrec,
Utilizzando il nostro sito, riconosci di aver letto e compreso le nostre Informativa sui cookie e Informativa sulla privacy.
Licensed under cc by-sa 3.0 with attribution required.