Launchpad e code review
Leonardo Pistone – lep – Agile Business Group

Agile Business Group - http://www.agilebg.com - info@agilebg.com
Chi è Agile BG?
●

Dedicata a OpenERP dal 2009

●

Primi progetti su OpenERP 5.0

●

Italia e Svizzera

Agile Business Group - http://www.agilebg.com - info@agilebg.com
Per chi è questo tutorial?
●

Per chi partecipa a OpenERP in Italia

●

Per chi vuole partecipare a OpenERP in Italia

●

Programmatori

●

Non programmatori

Agile Business Group - http://www.agilebg.com - info@agilebg.com
“Talk is cheap,
show me the code”
Linus Torvalds

Agile Business Group - http://www.agilebg.com - info@agilebg.com
“Come faccio a
partecipare?”

Agile Business Group - http://www.agilebg.com - info@agilebg.com
1. Perché?
2. Come?
Agile Business Group - http://www.agilebg.com - info@agilebg.com
1. Perché?
●

Win-win

●

Abbondanza

●

Somma > 0

●

Lo sforzo rimane nel tempo

●

Chi partecipa è un esperto

Agile Business Group - http://www.agilebg.com - info@agilebg.com
1. Stile libero
●

Molti esempi di successo

●

Regole diverse, confronti difficili

●

Attori diversi

●

Community Italiana

●

OCA

Agile Business Group - http://www.agilebg.com - info@agilebg.com
1. Perché?

https://code.launchpad.net/~icsergio/openobject-italia/remake-vat-registries/+merge/152598

Agile Business Group - http://www.agilebg.com - info@agilebg.com
1. Perché?
“Instead of imagining that our main
task is to instruct a computer what to
do, let us concentrate rather on
explaining to human beings what we
want a computer to do.”
Don Knuth
Agile Business Group - http://www.agilebg.com - info@agilebg.com
1. Perché?

“Code is read more
often than it is written”
Guido Van Rossum

Agile Business Group - http://www.agilebg.com - info@agilebg.com
1. Perché code review?
●

Altri umani leggono il diff

●

Bus factor

●

Leggibilità. Se non c'è, è un baco.

●

“Given enough eyeballs, all bugs are shallow” (Raymond)

●

Cultura sana e aperta: abitudine alla critica costruttiva

●

La discussione stessa rimane e dà valore aggiunto
http://alexgaynor.net/2013/sep/26/effective-code-review/

Agile Business Group - http://www.agilebg.com - info@agilebg.com
2. Come?
“Launchpad is a unique collaboration
and hosting platform for software
projects.”
http://launchpad.net/+tour

Agile Business Group - http://www.agilebg.com - info@agilebg.com
2. Come?

1. Utente
http://launchpad.net/+tour

Agile Business Group - http://www.agilebg.com - info@agilebg.com
Agile Business Group - http://www.agilebg.com - info@agilebg.com
2. Come?

2. Team
http://launchpad.net/+tour

Agile Business Group - http://www.agilebg.com - info@agilebg.com
Agile Business Group - http://www.agilebg.com - info@agilebg.com
2. Come?

2. Progetto

Agile Business Group - http://www.agilebg.com - info@agilebg.com
Agile Business Group - http://www.agilebg.com - info@agilebg.com
2. Come?

3. Baco

Agile Business Group - http://www.agilebg.com - info@agilebg.com
Agile Business Group - http://www.agilebg.com - info@agilebg.com
Come riportare un baco
●

Riprodurre

●

Da dove parto?

●

Cosa faccio?

●

Cosa ottengo?

●

Cosa mi aspetto?

●

Solo alla fine, facoltativo, che soluzione propongo?

●

In un mondo ideale: un test automatico

Agile Business Group - http://www.agilebg.com - info@agilebg.com
2. Come?

4. Branch

Agile Business Group - http://www.agilebg.com - info@agilebg.com
Agile Business Group - http://www.agilebg.com - info@agilebg.com
4. Branch
●

Proprietario

●

Progetto

●

Stato

●

Serie

●

Ultimo commit

●

Stacked branch

Agile Business Group - http://www.agilebg.com - info@agilebg.com
2. Come?

5. Merge proposal

Agile Business Group - http://www.agilebg.com - info@agilebg.com
Agile Business Group - http://www.agilebg.com - info@agilebg.com
5. Merge proposal
●

Da dove?

●

A dove?

●

Chi?

●

Chi approva?

●

diff

●

Commenti

Agile Business Group - http://www.agilebg.com - info@agilebg.com
2. Come?

6. Finale: Merge!

Agile Business Group - http://www.agilebg.com - info@agilebg.com
Agile Business Group - http://www.agilebg.com - info@agilebg.com
6. Finale: Merge!
●

bzr info

●

bzr status

●

bzr merge

●

bzr commit

●

bzr push

●

(caffé)

●

Effetti su baco, branch, MP

Agile Business Group - http://www.agilebg.com - info@agilebg.com
7. Direzione

Agile Business Group - http://www.agilebg.com - info@agilebg.com
8. Grazie!

Agile Business Group - http://www.agilebg.com - info@agilebg.com

Launchpad e code review

Editor's Notes

  • #2 Perché e come
  • #5 Cioè, il mio obiettivo è di indirizzare verso un modo di discutere proficuo, che produce software, invece di discussioni “economiche” che non portano a molto.
  • #8 Opinionated Non essere buoni Non buona volontà
  • #9 Opinionated Non essere buoni Non buona volontà
  • #10 Esempio positivo. Non mostro esempi negativi, ma possiamo pensare a lunghi thread un po' vaghi.
  • #11 Scriviamo per umani prima, per una macchina poi.
  • #12 → Qualità → Manutenzione → Comprensione
  • #14 Oltre a Launchpad: Github, bitbucket, sourceforge
  • #24 Oltre a Launchpad: Github, bitbucket, sourceforge
  • #25 Scriviamo per umani prima, per una macchina poi.
  • #28 Scriviamo per umani prima, per una macchina poi.
  • #30 Oltre a Launchpad: Github, bitbucket, sourceforge
  • #31 Scriviamo per umani prima, per una macchina poi.