intertionalization and text corrections on GitHub import.

[roipoussiere]

Ticket de référence : Aucun

Objet de la PR : amélioration des messages pour l'import GiHub :

• intertionalisation de la classe DownloadGithubService ;
• corrections de texte (ie : import sur GitHub -> import depuis GitHub) ;
• remplacé certains accents du fichier .properties en chaine utf-8.

[coveralls]

Coverage decreased (-12.7%) to 70.769% when pulling cd856d4 on roipoussiere:master into c367875 on firm1:master.

[firm1]

Bizarre celui-là

[roipoussiere]

Corrigé ici.

Comment je fais pour modifier ma PR ?

[coveralls]

Coverage decreased (-12.7%) to 70.769% when pulling bef32f6 on roipoussiere:master into c367875 on firm1:master.

[firm1]

Pas besoin, de modifier la PR. Pousser le commit la met a jour directement.

Le jeu. 22 sept. 2016 12:20, Nathanaël Jourdane notifications@github.com
a écrit :

Corrigé ici
roipoussiere@bef32f6
.

Comment je fais pour modifier ma PR ?

—
You are receiving this because you commented.

Reply to this email directly, view it on GitHub
#245 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFyPX3INSytkRuabk1V81CcVArBA2ouKks5qslZygaJpZM4KDuHa
.

[WinXaito]

Euh attend il y a le même code dans ton autre PR. tu as pas un petit problème ?

[roipoussiere]

Euh attend il y a le même code dans ton autre PR. tu as pas un petit problème ?

Non

[WinXaito]

Haaaa nanana un squash va faire 1 seul commit des 3 commits, pas supprimer les commits par rapport à cette PR.

Ou alors je me louse complétement @firm1. Mais selon moi le plus simple est de fermer cette PR et tout faire sur l'autre (Mais attend la confirmation de @firm1)

[roipoussiere]

Oui mais si il valide la PR précédente et qu'il fait un squash de celle-ci, la différence sera uniquement le dernier commit.

[firm1]

Bon j'avais pas bien compris la problématique. Je suppute que WinXaito soit
dans le vrai.
Quand on va merger l'autre, on verra bien le nouveau diff ici.

Le jeu. 22 sept. 2016 18:47, Nathanaël Jourdane notifications@github.com
a écrit :

Oui mais si il valide la PR précédente et qu'il fait un squash de
celle-ci, la différence sera uniquement le dernier commit.

—
You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub
#245 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFyPX84xolo7iD_l5xzx4Sa8ZJ_tSU_Dks5qsrEdgaJpZM4KDuHa
.

[WinXaito]

Donc cette PR n'aura plus d'utilité.

Et au final à quoi bon, ça ne change vraiment pas grand chose. Il ne faut juste pas que tu continues de travailler sur cette PR, sinon tu vas de faire ch**r sur le merge pour corriger les conflits :)

[roipoussiere]

Du coup on peut intégrer la PR là (avec un squash) ? C'est quoi le soucis ?

[WinXaito]

En gros tu as la modif de l'internalisation GitHub sur cette PR et sur l'autre PR.

Ce qu'on appel un squash, c'est par exemple sur ta PR pour modifier hauteur minimal et largeur minimal, tu as 3 commits. Lors du merge tu peux mettre ces 3 commits dans 1 seul. C'est un squash, mais on peut pas faire de l'inter-pr

[roipoussiere]

Oui mais pour cette PR là je ne vois pas le problème. C'est indépendant de l'autre vu que je l'ai faite avant.

[WinXaito]

Hm ça veut dire qu'en fait tu as fait l'autre à partie de celle-ci sans te repositionner sur la master. Il faudra voir au moment tu merge ce que ça donne. Mais à mon avis il y aura un conflit à résoudre (Ce qui n'est pas bien grave).

[firm1]

Z'inquietez pas, au pire je m'occupe de corriger les conflits. Bossez
normalement.

Le jeu. 22 sept. 2016 21:28, Kevin Vuilleumier notifications@github.com a
écrit :

Hm ça veut dire qu'en fait tu as fait l'autre à partie de celle-ci sans te
repositionner sur la master. Il faudra voir au moment tu merge ce que ça
donne. Mais à mon avis il y aura un conflit à résoudre (Ce qui n'est pas
bien grave).

—
You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub
#245 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFyPXwj7U68FF73ulGCsTGfZQPJNWzndks5qstbegaJpZM4KDuHa
.

[firm1]

Thanks, c'est ok donc c'est mergé.