Closed SpaceFox closed 10 years ago
A faire sur la branche d'intégration, je pense.
Je veux bien relire et corriger, mais elle est où cette fameuse branche d'intégration ?
https://github.com/Taluu/ZesteDeSavoir/tree/integration
2014-05-14 8:10 GMT+01:00 victor felder notifications@github.com:
Je veux bien relire et corriger, mais elle est où cette fameuse branche d'intégration ?
— Reply to this email directly or view it on GitHubhttps://github.com/Taluu/ZesteDeSavoir/issues/522#issuecomment-43049100 .
Dans la looonnngue liste des branche, il y a la petite "intégration" qui se promène là.
Par contre je n'ai rien push depuis quelque jour (oubli...) et donc tu n'auras pas le truc tout à fait à jour, mais pour les corrections ça ne devrait pas poser de problème. Au pire les conflits seront pour moi :')
Dans la looonnngue liste des branche, il y a la petite "intégration" qui se promène là.
Il faudra faire un peu de ménage d'ailleurs, je pense qu'on en a une bonne moitié qui ne sert plus à rien.
Le 14 mai 2014 10:07, DEMODE Alexandre notifications@github.com a écrit :
Dans la looonnngue liste des branche, il y a la petite "intégration" qui se promène là.
Par contre je n'ai rien push depuis quelque jour (oubli...) et donc tu n'auras pas le truc tout à fait à jour, mais pour les corrections ça ne devrait pas poser de problème. Au pire les conflits seront pour moi :')
— Reply to this email directly or view it on GitHubhttps://github.com/Taluu/ZesteDeSavoir/issues/522#issuecomment-43053118 .
Hum, me semblait pourtant avoir bien fouillé cette longue liste de branches. Au temps pour moi.
Dans le doute tu as sur Github un champ de recherche quand tu déroule la liste des branches.
Ou sinon : git branch | grep "integration"
et hop le tour est joué \o/
Dans le doute tu as sur Github un champ de recherche quand tu déroule la liste des branches.
Ou sinon :
git branch | grep "integration"
et hop le tour est joué \o/
Alex, sans trop m'avancer, je pense que @vhf sait se servir de git et de github hein !
Je disais ça au cas où ^^ on a vite fait d'oublier ce genre de petite feature pratique :)
Effectivement, je me sers de Github tous les jours. J'ai juste pas trouvé la branche d'intégration et m'en excuse.
Au passage Alex-D, je commencerais volontiers la relecture/correction mais j'ai encore et toujours ces problèmes d'EOL (CRLF vs LF) sur ta branche. Vu que c'est corrigé en un coup de cuiller à pot, je l'aurais volontiers fait et pushé, mais ça t'aurait fait des milliers de conflits, donc soit j'attends que tu push tes modifs et je corrige les EOL, soit tu push tes modifs et tu corriges les EOL. DIs-moi ce que tu préfères.
Je n'ai pas push le "patch" peut-être. Du coup, je te push ça se soir, attends pour l'instant sinon ça va encore tout casser =D
Chez moi tout est déjà corrigé en fait, j'ai juste oublié de push la dernière fois ><"
Merci en tous cas :)
Vu que c'est pas évident de le trouver ni de le savoir hors-contexte, le mot "Réservé" dans le template d'administration des tutos doit s'écrire "Réserver".
Du coup ca en est ou ca ? Car ca fait partie des choses sur lesquels mon niveau de noob pourrait probablement traiter, mais est-ce le moment le plus opportun ? (car tant que des choses vont bouger au niveau integration ca risque d'etre chiant a chaque commit...)
Tu peux corriger dans ma branche sans que ça pose trop de soucis.
Il y a une autre discussion je ne sais trop où autour de l’internationalisation et tout ça et ça rejoint ce sujet.
J'avais cru comprendre que Victor était sur le coup. Je pensais le faire aussi (avec mon niveau de noob en django, c'est un truc qui m'allait bien).
Dans l'absolu, vous pouvez avancer en parallèle, si vous commitez bien vos modifications régulièrement sur ma branche, ça va le faire. Voire, créez une branche de correction chacun que vous venez PR sur la mienne petit à petit, histoire de rester propres et éviter de tout casser :)
Si on est plusieurs a la correction il va falloir s'organiser pour pas dupliquer le travail...
Y en a un qui fait le module tuto, l'autre article, l'autre forum. C'est pas du texte qu'il manque.
Par contre, il y a des textes dans les templates et dans les vues.
Je laisse volontiers ma place. On May 22, 2014 2:40 PM, "Eskimon" notifications@github.com wrote:
Si on est plusieurs a la correction il va falloir s'organiser pour pas dupliquer le travail...
— Reply to this email directly or view it on GitHubhttps://github.com/Taluu/ZesteDeSavoir/issues/522#issuecomment-43882366 .
Bon allez, je me prends le module tuto, il faut bien commencer quelque part... Je fork la branche intégration et essairai de faire des PR régulières si je vois qu'il y a beaucoup de corrections à faire (et une seule grosse s'il y avait pas grands chose).
Voilà une première passe de faite sur les tutos. Il en reste peut-être mais j’espère pas...
Quelques remarques pour une harmonisation entre les corrections:
Accents sur majuscules sur la première lettre de la phrase: Éditer Points à la fin de phrase ? Faudrait trouver une règle... Franglais : offline -> hors-ligne beta -> bêta
Vu les differents merge qui ont eu lieu récemment, on peut continuer sur la branche dev ou il faut toujours faire ca sur "intégration" ?
branche dev.
Le 4 juin 2014 17:31, Eskimon notifications@github.com a écrit :
Vu les differents merge qui ont eu lieu récemment, on peut continuer sur la branche dev ou il faut toujours faire ca sur "intégration" ?
— Reply to this email directly or view it on GitHub https://github.com/zestedesavoir/zds-site/issues/522#issuecomment-45105992 .
Je sais pas si la version que je regarde sur la prod' est la bonne mais sur la capture suivante :
Le mot "Réservé" devrait s'écrire "Réserver" puisque l'article n'est pas encore réservé...
Je sais qu'il y avait eu discussion la dessus avec @Alex-D ;)
Ca se peut qu'on ait corrigé à un endroit mais pas un autre, y a beaucoup de pages ^^
Fais-toi plez' pour les corrections :)
Je profite de cette issue pour faire remonter les soucis lies a l'orthographe...
il manque un mot dans le message : Ce sujet est résolu, l'auteur de ce sujet a trouvé une solution satisfaisant A son problème.
Non, c'est soit :
satisfaisant son problème
soit
satisfaisante à son problème
Ce qui est différent ; en plus il manquait un accent sur le À
.
Perso je vote pour la deuxième
Ce sujet est résolu, l'auteur de ce sujet a trouvé une solution satisfaisant à son problème.
Et pourquoi pas simplement "Ce sujet est résolu, l'auteur de ce sujet a trouvé une solution à son problème." ?
Le 12 juin 2014 10:06, Eskimon notifications@github.com a écrit :
Perso je vote pour la deuxième
Ce sujet est résolu, l'auteur de ce sujet a trouvé une solution satisfaisant à son problème.
— Reply to this email directly or view it on GitHub https://github.com/zestedesavoir/zds-site/issues/522#issuecomment-45840534 .
+1 Coyote. "solution satisfaisant(e)" ça fait un peu pléonasme en fait.
Pourquoi pas en effet. Dans tous les cas, la syntaxe est bonne, que celui/celle qui le souhaite fasse une PR :)
Le verbe "satisfaire" est transitif direct.
« Une solution satisfaisant à son problème » est tout sauf français. La formulation actuelle (« une solution satisfaisant son problème ») est tout à fait correcte grammaticalement parlant.
Sinon +1 pour « une solution à son problème ». C'est plus léger.
On dirait qu'elle est perturbante pour ceux qui ne maitrisent pas les rouage de la langue française.
Le 12 juin 2014 11:02, ArnaudCalmettes notifications@github.com a écrit :
Le verbe "satisfaire" est transitif direct.
"Une solution satisfaisant à son problème" est tout sauf français. La formulation actuelle ("une solution satisfaisant son problème") est tout à fait correcte grammaticalement.
— Reply to this email directly or view it on GitHub https://github.com/zestedesavoir/zds-site/issues/522#issuecomment-45845438 .
Si on veut éviter toute emmerde et toute répétition (résolu / solution, etc.), on écrit uniquement L'auteur de ce sujet a trouvé une solution à son problème.
et c'est marre.
C'est clair, concis et efficace.
L'avantage de répéter résolu, c'est pour le SEO aussi.
"Résolu" étant déjà présent dans le titre de la page, je pensais que c'était suffisant. My bad.
C'est vrai que je l'ai remonté dans le titre... Du coup on prendre la solution :
L'auteur de ce sujet a trouvé une solution à son problème.
Et ça convient à tout le monde ?
Va pour ça.
Voilà qui est fait dans la PR
Faudra penser à mettre un "D" majuscule à "discussions, débats, ... " dans la description du forum "Bar à smoothies".
Ca c'est pas dans le code pour le coup, ping @firm1 :)
Si vous avez besoin d'un coup de main pour la correction, je suis présent. Il suffit de me dire quoi corriger. Je me suis déjà permis de faire une PR :)
Ça a été fait non ?
Au moins partiellement. Je n'ai aucune idée de si tout a été relu ou non. Je pense qu'au pire les coquilles restantes nous seront signalées.
Oui, je pense que ça peut être fermé. S'il y a des fautes, elles sont de toute façon signalées et il y a toute de suite une PR.
ProTIp : Monitorer le sujet "Les petits pixels" qui généralement les vois apparaître.
Yop,
J'utilise pour la première fois mon compte GitHub, donc je vais juste me permettre de faire mes propositions et signaler quelques corrections.
Pour le message à la première mise en beta : "J'aimerai avoir…" -> "J'aimerais avoir…" "Si vous êtes interessé…" -> "Si vous êtes intéressé…" (à deux endroits, lignes 615 et 675 de views.py)
Autrement, comment est choisi l'auteur du message dans le forum ? C'est celui qui a créé le tutoriel, c'est celui qui a demandé la mise en beta ?
Quid des co-rédacteurs ? Il serait peut-être sympa de les mentionner aussi. Je pense avoir inauguré le système, donc je me permets de lier le sujet où j'ai d'ailleurs apporté les corrections signalées ci-dessus et adapté pour y inclure mon camarade.
Autrement, comment est choisi l'auteur du message dans le forum ? C'est celui qui a créé le tutoriel, c'est celui qui a demandé la mise en beta ?
Là tu peux ouvrir un sujet sur le forum, c'est plus tellement le sujet ici.
Vérifier les textes et en corriger l'orthographe.