a4-dev team mailing list archive

[Andrea Corbellini <corbellini.andrea@xxxxxxxxx>]

On Tue, 2010-10-05 at 23:34 +0200, Andrea Gualano wrote: 
> Non ho alcuna esperienza in nessuno degli strumenti citati, quindi mi 
> fido ciecamente del vostro giudizio.
> 
> Ho però un po' di osservazioni generali:
> 1) Ognuno è libero ovviamente di usare gli strumenti che vuole, ma...
> 2) decidiamo quali regole di stile deve rispettare il codice prima di 
> essere mergiato. Queste regole devono essere validabili con uno 
> strumento automatico (decidiamo quale). Ciascuno dovrà cercare di 
> ricordarsi di far girare tale strumento prima di proporre il merge.
> Ovviamente i casi in cui lo strumento "sbaglia" andranno trattati con 
> ragionevolezza.

Per quanto riguarda questi due punti, credo di aver risposto nella mail
che ho inviato qualche minuto fa.

> 3) Visto che nessuno qui ha tempo da perdere, sarebbe molto preferibile 
> se lo strumento in questione fosse in grado di risolvere le violazioni 
> invece di limitarsi a lamentarsi. Questo ovviamente si applica 
> principalmente a questioni di formattazione più che a questioni di lint.

Non conosco alcun tool che fa questo. :-(

> 4) Per quanto riguarda la formattazione del codice, la mia esperienza è 
> che adottare un formattatore automatico (impostato allo stesso modo per 
> tutti) non solo rende più semplice la lettura del codice, ma riduce di 
> molto il rumore nel repository, rendendo più facile capire le patch e 
> individuare commit interessanti. Mi piacerebbe quindi che adottassimo 
> ufficialmente uno strumento per la formattazione automatica del codice.

Questo credo sia difficile da fare: credo che quasi ognuno usi un editor
diverso. Ma forse ho capito male quello che intendi, puoi citare il nome
di qualche strumento?

> ciao,
> A.
> 
> _______________________________________________
> Mailing list: https://launchpad.net/~a4-dev
> Post to     : a4-dev@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~a4-dev
> More help   : https://help.launchpad.net/ListHelp

-- 
Andrea Corbellini
Ubuntu Member  | http://www.ubuntu.com
BeeSeek Member | http://www.beeseek.org

Attachment: signature.asc
Description: This is a digitally signed message part