implementation of travisci

[foutoucour]

No description provided.

[foutoucour]

Faut rajouter travis maintenant :)

[foutoucour]

Et faire la PR de tes tests :)

[nzin]

ca m'a l'air bon. tu veux que je merge?

[foutoucour]

@nzin il faut rajouter l'integration travis. Meme s'il y a pas de tests encore :)

[foutoucour]

Je crois que tu vas me detester avec la mise en place de golint :)

[nzin]

Salut,

le truc c'est que j'utilise systématiquement Visual Code Studio, et il me lint automatiquement.
Si tu mets golint: soit il fait exactement le boulot, soit ca va rentrer en conflit.

[nzin]

bref, je ne vois pas trop l'intérêt

[foutoucour]

je vois. Mais je suis surpris parce golint trouve beaucoup de points a revoir sur le code. Tu as quoi comme setup dans VCS?

[nzin]

Aucune idée, je peux regarder. mais visual code studio m'a proposé plusieurs plugins et je les ai tous installé

…

On Wed, Jan 10, 2018 at 10:43 AM, Jordi Riera ***@***.***> wrote: je vois. Mais je suis surpris parce golint trouve beaucoup de points a revoir sur le code. Tu as quoi comme setup dans VCS? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#5 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAH4keZzq5myPjDCrGpM7B9Kl62iXZx5ks5tJNqggaJpZM4RVdWK> .

[foutoucour]

je merge? :)

[nzin]

est ce qu'on pourrait closer ce PR et en faire un autre? je vois des redondance avec ce qui a déjà été fait.
genre: docker-compose run --rm app go test ./... ou bien le Dockerfile

[nzin]

There is already a Dockerfile, this PR isn't redundant?

[nzin]

je me suis trompé, je croyais qu'on était sur dctycoon...

[nzin]

LGTM