5 Jul
2012
5 Jul
'12
9:37 a.m.
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