Maj doc pour v1.0 #1488
Maj doc pour v1.0 #1488
Conversation
|
Ca me semble bon tout ca (mais il faut faire un duplicata pour sphinx ?) |
|
D'accord avec Eskimon, mais à mon avis, on fera ça d'un coup quand on aura fini de discuter par ici (ou en sois, tout le monde est d'accord sur le principe) |
| @@ -41,9 +47,10 @@ Les contributions externes sont les bienvenues ! | |||
| | Nouvelle Fonctionnalité ? | [oui|non] | |||
| | Tickets concernés | [Liste de tickets séparés par des virgules] | |||
| ``` | |||
| * Ajoutez des notes de QA (Quality Assurance). Ces notes doivent permettent à un testeur de comprendre ce que vous avez modifié, ce qu'il faut tester en priorité et les pièges auxquels il doit s'attendre et donc sur lesquels porter une attention particulière. | |||
There was a problem hiding this comment.
Y'aurais moyen de rajouter un "précisez tout particulièrement s'l est nécéssaire d'effectuer un python manage.py migrate, python manage.py loaddata fixture/*.yaml ou gulp build" ? J'avoue que j'y pense pas forcément et que c'est pas toujours nécéssaire non plus.
|
Du reste, ça me semble bon. Par contre, je vous pas d'ou viennent les merge conflicts :o |
|
Ils viennent certainement de la pep8 + màj de doc de @firm1 cette nuit qui a mis en conflit quasi toutes les PR. |
|
Me doutes, ce qui m'étonne un peu plus, c'est que ça devrait revenir avec les MàJ. Spacefox, à mon avis, t'as oublié de mettre à jour tes remotes (et à mon avis, t'aura un commit bidon pour régler les conflits). |
Je les ai mises à jour hier soir, avant le commit de @firm1, et j'ai rebase cette branche dessus. Je regarderai ce soir. |
|
Faut que dans ton outil tu arrive à faire l'équivalent de ça : Tu auras des conflits sur le README je pense, vu qu'il a mis à jour la ligne pour flake8 et que toi tu l'a déplacé. |
|
C'est juste très exactement ce que j'ai fait hier hein. C'est juste qu'il faut que je le refasse aujourd'hui. |
Eskimon
added
C-Back
SpaceFox
force-pushed
the
maj_doc_pour_v1.0
branch
from
c3260e3
to
e8fd901
Compare
|
Et voilà ! |
|
Moi je dis à merger :) |
QA : C'est une amélioration de la doc, donc pas grand chose