Le 05/07/2012 10:25, Cédric Houbart a écrit :
> Bonjour
Bonjour,
> J'ai ajouté l'import d'entrées pas csv pour supporter l'import des données de
> banques (on reçoit notre relevé de banque et on l'injecte dans le journal de
> banque). J'en ai également profité pour mettre à jour un peu la javadoc des
> méthodes d'import et quelques nettoyages de code. Le patch est en pièce jointe.
Votre patch me semble tout à fait correct.
Désirer vous avoir directement les droit de commit sur le projet pour que
vos contributions soient intégrées plus rapidement ?
Si oui, il vous faudra un compte sur la forge http://www.chorem.org/,
je vous donnerai les droits ensuite.
>
> Aussi est-ce que lancer les diff par mails est la bonne méthode et est-ce que
> vous avez une convention de code quelque part et des pratiques de découpage
> des commits ? :)
>
> Notamment pour le code si les commentaires sont en Français ou Anglais et quel
> indentation pour les instructions sur plusieurs lignes (8 espaces
> d'indentation, alignement à la parenthèse…), j'ai vu plusieurs méthodes à
> plusieurs endroits.
Globalement, les règles de codages sont celles recommandées par la norme
Java (préconisée par oracle):
- les parenthèse ouvrantes sur la même ligne
Globalement, la configuration par défaut des IDE.

A coté de ça:
- le code doit être en anglais
- les commentaires doivent, si possible, être en anglais, mais nous préférons
un bon commentaire en français qu'un mauvais en anglais ou pas de commentaire
du tout
- les indentations doivent être sur 4 caractères (et non sur des tabulations)
(les instructions multilignes sont parfois sur 8 caractères car eclipse fait
comme ça
mais elle devrait être sur 4).

-- 
Éric Chatellier <chatellier@codelutin.com>
Tel: 02.40.50.29.28
http://www.codelutin.com