Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Correctifs sur la recette des cookies #2508

Merged
merged 1 commit into from
Apr 13, 2015

Conversation

SpaceFox
Copy link
Contributor

@SpaceFox SpaceFox commented Apr 6, 2015

Quelques correctifs sur la recette des cookies (qui était moins éloignée de la "bonne" que dans mon souvenir) :)

@Eskimon
Copy link
Contributor

Eskimon commented Apr 6, 2015

C'est une correction de bug ou une évolution ? :-°

@SpaceFox
Copy link
Contributor Author

SpaceFox commented Apr 6, 2015

Un peu les deux : une évolution sur les proportions et une correction de bug sur la température du four :)

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling af5da37 on SpaceFox:fix-cookies into db70edb on zestedesavoir:dev.

@@ -113,11 +113,11 @@
<p>{% trans "Pour 30 cookies environ (parce que c'est bon, on va en faire plein" %} !) :</p>
<ul>
<li>{% trans "500 g de farine" %}</li>
<li>{% trans "500 g de chocolat à pâtisser" %}</li>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ça me parait beaucoup ! Perso je laisserais 300 g

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Haha non, je te garantis que ma recette c'est bien 500 g :)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK. J'ai du faire de trop gros morceaux. C'est dur de couper le chocolat je trouve ! :D

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oui, le "bien aiguisé" est important.

@Situphen
Copy link
Member

Situphen commented Apr 6, 2015

Perso, je sais pas si mes parents vont me laisser faire la QA ! Risque d'overdose... :D

@pierre-24
Copy link
Member

Pareil, j'ai donné pour la QA précédente :o

@Eskimon
Copy link
Contributor

Eskimon commented Apr 7, 2015

La QA c'est un don se sa personne avant tout, c'est un sacrifice personnel pour rendre service a tout ceux qui suivront !

@Eskimon
Copy link
Contributor

Eskimon commented Apr 7, 2015

Je pense que pour cette QA une fournée spécial Assemblée Générale de l'association serait une bonne méthode de test. On aurait un échantillon de testeur varié comme ca :)

@pierre-24
Copy link
Member

Oh, dieu, ça me fait presque regretter de ne pas faire partie de l'association :p

@Situphen
Copy link
Member

Situphen commented Apr 7, 2015

Oh, dieu, ça me fait presque regretter de ne pas faire partie de l'association :p

Moi aussi :D

@firm1
Copy link
Contributor

firm1 commented Apr 7, 2015

+1

@Eskimon
Copy link
Contributor

Eskimon commented Apr 11, 2015

Ce soir on aura ptet un beta-test + un "bon pour merge" de la part de l'association ?

@SpaceFox
Copy link
Contributor Author

Y'aura un correctif au moins sur le temps de préparation avant de pouvoir merge ;)

@SpaceFox
Copy link
Contributor Author

Voilà, le texte correspond exactement à ce qui a été QA pendant l'AG :)

Si ça vous va, y'a plus qu'à merger.

@landscape-bot
Copy link

Code Health
Repository health increased by 1% when pulling 186e68e on SpaceFox:fix-cookies into bba7dd4 on zestedesavoir:dev.

@pierre-24
Copy link
Member

C'est bien, Landscape trouve que la recette améliore la santé du code ^^

@Situphen
Copy link
Member

C'est bien, Landscape trouve que la recette améliore la santé du code ^^

C'est parce qu'on code mieux le ventre plein :D

@SpaceFox
Copy link
Contributor Author

WTF !?

@pierre-24
Copy link
Member

Quand je vous disais que ces outils font des choses un peu étranges ^^

(et encore, attend de voir Travis qui va t'inventer une diminution de couverture)

<p>{% blocktrans %} On peut faire des cookies à beaucoup de parfums, mais le <em>must</em> reste le cookie au chocolat" {% endblocktrans %}.</p>
<p>{% trans "Pour découper le chocolat, vous aurez besoin d'un grand couteau solide (donc pas un couteau céramique) et très bien aiguisé. Non, le vôtre n'est pas assez aiguisé. Attaquez la plaque de chocolat côté lisse (carrés sur le dessous), c'est plus simple." %}</p>
<p>{% trans "La cuisson parfaite se joue à quelques secondes près (sérieusement), alors surveillez-les bien ! Un SMS de trop, et vous obtenez des cookies en béton. Ce qui serait dommage. Les cookies sont assez mous en sortie de four, c'est normal : ils durcissent en refroidissant." %} !</p>
<p>{% blocktrans %} On peut faire des cookies à beaucoup de parfums, mais le <em>must</em> reste le cookie au chocolat{% endblocktrans %}.</p>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lors du merge de release-v1.7 dans dev, il faudra veiller à laisser cette phrase "Bien qu'avec un peu de noix de coco ça ne se refuse pas !" amenée normalement par le commit fab4daa !

@SpaceFox
Copy link
Contributor Author

Bref, c'est mergeable non ?

@pierre-24
Copy link
Member

Toutafé ;)

@pierre-24
Copy link
Member

Bon forçons le destin: ping @Eskimon ?

GerardPaligot added a commit that referenced this pull request Apr 13, 2015
Correctifs sur la recette des cookies
@GerardPaligot GerardPaligot merged commit 839a02f into zestedesavoir:dev Apr 13, 2015
@GerardPaligot
Copy link
Member

Merci à tous pour vos contributions, je merge.

@GerardPaligot GerardPaligot added this to the Version 1.8 milestone Apr 13, 2015
@Eskimon
Copy link
Contributor

Eskimon commented Apr 14, 2015

han grillé par le robot !

@SpaceFox SpaceFox deleted the fix-cookies branch April 14, 2015 06:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants