Git: come disporre di permessi misti per unire una richiesta pull in un ramo protetto?

Question

Git: come disporre di permessi misti per unire una richiesta pull in un ramo protetto?

#1 da (3 voti)

1

So che, su github, puoi definire un ramo protetto e consentire solo a determinati utenti di unire una richiesta pull qui.

Supponiamo, ad esempio, che gli utenti A, B, C e D siano sviluppatori e A sia team leader e quello che dovrebbe essere responsabile della fusione delle richieste di pull nel ramo di sviluppo protetto. Succede che A è anche uno sviluppatore e tra tutte le altre attività, a volte fa qualche codifica ed emette una richiesta di pull.

Le richieste di pull non devono essere approvate nel brach di sviluppo dall'autore, quindi A non può approvare una richiesta di pull che ha creato.

Quindi, potrei aggiungere al ramo di sviluppo protetto un insieme di regole di permesso? La mia idea era che entrambi gli utenti A e B potevano approvare fusioni nel ramo di sviluppo protetto ma A sarebbe in grado di approvare richieste di pull da B, C e D. Mentre B sarebbe in grado di approvare solo le richieste di pull da A.

Spero che la mia domanda sia stata abbastanza chiara. È fattibile?

git github pull-requests

posta gtludwig 31.01.2017 - 10:50

fonte

1 risposta

Leggi altre domande sui tag git github pull-requests

La modifica del membro privato di una classe tramite un operatore di pedice sovraccarico interrompe l'incapsulamento? Parametri dimagranti per un metodo

score 3 · Accepted Answer

Non c'è un processo predefinito qui, nessun modo standard del settore.

L'importante non è tanto chi approva le richieste di pull, ma il codice che viene unito viene riesaminato? Affinché una persona approvi le richieste di pull, quella persona sarà responsabile delle recensioni del codice del codice in questione. Tuttavia, sollevi un punto valido: se il codice della persona A non viene esaminato dai peer, come possiamo garantire la qualità del codice che viene unito?

Questo si avventura nel territorio basato sull'opinione pubblica, ma questo è ciò che ha funzionato meglio nella mia esperienza:

Tutto il codice che viene unito al "ramo del gold standard" o qualsiasi altra cosa tu voglia chiamarlo, dovrebbe essere esaminato dal team intero indipendentemente da chi sia l'autore.

Sono un ingegnere del software con esperienza e tende a scrivere un buon codice (beh, quando lo controllo almeno). Tuttavia, ho fatto molti errori. Alcuni di loro sono stati presi in revisione del codice, altri no. Tuttavia, quando sono in una squadra che fa recensioni di codice, la maggior parte degli errori, miei o di altri, tendono a essere catturati prima che finiscono con il controllo della versione.

In altre parole: la persona A è quella che esegue effettivamente la richiesta di pull, ma l'intero team rivede il codice insieme prima che la richiesta di pull sia approvata.