Etichetta per il ripristino del lavoro altrui

8

Recentemente ho avuto una discussione con un compagno di squadra che stavo "non consultandoli prima di tornare" perché li fa "sembrare un idiota". (Per contesto, questo è un progetto universitario e il ripristino è stato eseguito tramite il posizionamento del codice.)

Questo mi fa domandare: qual è la norma per informare il committer che deve essere fatto un ripristino? Come si dice loro che è necessario ripristinare questo commit senza istigare i sentimenti difficili?

    
posta tyteen4a03 11.02.2017 - 02:37
fonte

5 risposte

11

Avere recensioni del codice.

Questo dovrebbe risolvere la maggior parte di loro, se stai usando un commit puoi comunque effettuare richieste pull anche se è solo per un'altra persona che deve rivedere il tuo team.

Se gli errori lo fanno, allora è colpa della squadra e non solo di una persona.

Se non vuoi farlo per qualche motivo, non c'è un modo semplice per farlo. Non hai alcun processo per ottenere il codice nel repository, quindi, beh, non essere sorpreso quando i sentimenti delle persone si fanno male.

Come spesso accade, quando incontri situazioni in cui fai cose che frustrano gli altri, normalmente è bene parlare con loro prima di fare l'azione. In questo caso, probabilmente significa parlare con il tuo collega e chiedere "Sto pensando di ripristinare X a causa di [ragioni] - cosa ne pensi?"

    
risposta data 11.02.2017 - 04:29
fonte
3

O hai avuto un buon motivo per annullare il commit, o no. Se avessi una buona ragione, allora "mi fa sembrare un idiota" non è una buona controargomentazione. Detto questo, sarebbe educato informare la persona prima sui motivi per evitare argomenti.

E tutto ciò detto, sarebbe molto migliore per avere recensioni di codice, quindi speriamo che il ripristino non avvenga mai - potrebbe accadere che un commit venga rifiutato, ma non si dovrebbe entrare nel situazione in cui è ripristinata.

    
risposta data 12.02.2017 - 14:29
fonte
2

La domanda principale su questo argomento è "chi possiede il componente a livello tecnico"?

Se non c'è una risposta a questo, o la risposta è "noi tutti lo facciamo" o "nessuno lo fa" o hai solo un aspetto perplesso e nessuno vede un problema, non sprecare le tue energie e trovare un altro lavoro.

Se c'è una risposta e i colloqui con il commutatore non hanno portato a un accordo, mëet con il proprietario e il commutatore, fai il tuo caso e lascia decidere al proprietario.

La responsabilità condivisa è alla base di tutte le basi di codice deplorevoli e di dipendenti miserabili.

    
risposta data 11.02.2017 - 09:17
fonte
1

Penso sia giusto chiedere di essere consultato se qualcuno ha un investimento di tempo significativo in qualcosa, specialmente se è piuttosto recente. È meglio arrivare a un consenso prima. Idealmente dovresti convincere la persona a fare il revertimento.

Se ti imbatti in un disaccordo che non riesci a trovare, consulta una terza persona. Lascia che la maggioranza vota, o assicurati che la terza persona sia uno sviluppatore principale.

A volte devi lasciare andare le cose, non devi vincere ogni battaglia. Assicurati di essere stato ascoltato.

Penso che sia salutare non diventare attaccato al codice che scrivi. Ho ripristinato il mio codice abbastanza spesso, altri dovrebbero essere in grado di fare lo stesso quando appropriato.

    
risposta data 11.02.2017 - 12:39
fonte
1

Non tornare indietro, lascia che eseguano il ripristino.

Tipici motivi che puoi dare quando richiedi un ripristino:

  • "Your Commit XYZ rompe questo e quello usecase / functional / testcase"
  • "Your Commit XYZ non rispetta i nostri standard di codifica, ad es. indentazione errata"
  • "Il tuo commit XYZ viola i nostri processi organizzativi, ad esempio il commit del componente A deve essere rivisto prima del commit dal proprietario del componente B"

Solo motivi per ripristinare qualcosa non fatto da te

  • La persona viene estesa (vacanza / malattia)
  • Il commit ha interrotto la compilazione o fa qualcosa che impedisce ad altri membri del team di svolgere il proprio lavoro.
  • Il commit buggato è già in produzione e deve essere risolto al più presto
risposta data 12.02.2017 - 15:18
fonte

Leggi altre domande sui tag