Ajout d'un template de pull-request

[sandhose]

Q	R
Correction de bugs ?	non
Nouvelle Fonctionnalité ?	~non
Tickets (issues) concernés	-

OMFG ENFIN GITHUB AJOUTE UNE FEATURE UTILE

Hm, pardon.

cf. https://github.com/samccone/issue-and-pr-templates

[DevHugo]

o/ ça c'est méga bien.

[firm1]

Super.

Il manque peut-être un titre pour préciser les notes de QA.

Du coup, on peut le virer du fichier Contributing.md dans cette PR ?

[Emeric54]

Pourquoi ? On ne perd rien à le laisser.

[sandhose]

@firm1: je pense pas que ça remplace ce qu'il y a dans le CONTRIBUTING.md à ce propos, et que y'a aucune raison qu'on le vire. J'avais rapidement demandé sur IRC si je devais mettre les notes QA, on m'a dit que c'était pas la peine ; et puis de toutes façons, c'est pas ça qui est lourd à taper dans le message de la PR.

[AmauryCarrade]

Ce n'est pas lourd à taper, mais c'est aussi une incitation claire à le faire.

[firm1]

Yop l'idée c'est que ça t'incite à le faire. Parce que je ne suis pas sur que tout le monde lit le contributing jusqu'au bout. Après ça reste mieux que rien

[gustavi]

Personnellement j'aurai fait évolué la cartouche comme il suit :

| Q                                   | R
| ----------------------------------- | -------------------------------------------
| Correction de *bug* ?               | oui/non
| Nouvelle fonctionnalité ?           | oui/non
| Amélioration d'une fonctionnalité ? | oui/non
| Ticket(s) (_issue(s)_) concerné(s)  | (ex #1337)

### QA

* Instruction 1
* Instruction 2
* etc

[pierre-24]

Amélioration d'une fonctionnalité ? | oui/non

L'amélioration d'une fonctionnalité n'est-elle pas elle même une fonctionnalité ?

[gustavi]

Question intéressante ! Vous avez 4h.

[orandin]

Dans le cas des PR de mise à jour de la documentation, ne manque t-il pas une ligne comme il suit ?

| Mise à jour de la documentation ? | oui/non

[gustavi]

@SpaceFox ton avis ?

[SpaceFox]

KISS : vues le nombre, on peut se passer de l'amélioration de la doc dans le modèle.

En fait je remplacerais le cartouche par un truc du genre :

Q	R
Type de modification	correction de bug / nouvelle fonctionnalité / évolution
Ticket(s) (issue(s)) concerné(s)	(ex #1337)

Au final, garder le modèle de QA me semble utile.

[GerardPaligot]

@sandhose Est-ce que tu peux mettre à jour cette PR selon le commentaire de @SpaceFox ? :)

[gustavi]

Du coup je pense qu'on peut partir sur :

| Q                                   | R
| ----------------------------------- | -------------------------------------------
| Type de modification                | correction de bug / nouvelle fonctionnalité / évolution
| Ticket(s) (_issue(s)_) concerné(s)  | (ex #1337)

### QA

* Instruction 1 (ex : lancez `python manage.py migrate` et `npm run gulp build`)
* Instruction 2
* etc

ce qui donne :

Q	R
Type de modification	correction de bug / nouvelle fonctionnalité / évolution
Ticket(s) (issue(s)) concerné(s)	(ex #1337)

QA

• Instruction 1 (ex : lancez python manage.py migrate et npm run gulp build)
• Instruction 2
• etc

[pierre-24]

Moi j'aprouve ! :)

[artragis]

c'est toujours pas mergé ça?

[GerardPaligot]

@artragis Parce que la PR n'a toujours pas été mise à jour. :)

[Situphen]

Une nouvelle fonctionnalité est une évolution, non ?

[sandhose]

C'est bon :)

[GerardPaligot]

@sandhose Tu sais nous squashé ça et rajouter "[ci skip]" au début du commit pour éviter de lancer travis sur ta PR pour merger ça rapidement ? :)

[artragis]

a-ton vraiment besoin du CI-SKIP?

[SpaceFox]

Vu comment Travis galère en général, c'est pas un mal de lui éviter du taff inutile. Et c'est toujours ça de fait pour la planète :)

[GerardPaligot]

a-ton vraiment besoin du CI-SKIP?

On en a besoin parce que ça serait bien d'avoir un squash sur cette branche et de ne pas lancer Travis inutilement une fois le squash fait. :)

[SpaceFox]

@sandhose tu auras le temps de t'en occuper, ou tu veux qu'on le fasse ?

[sandhose]

Voila voila ! Désolé pour l'attente c:

[gustavi]

Bon je crois que tout le monde est OK donc je merge, merci @sandhose et tout le monde ! J'ai l'impression que GH se bouge un peu niveau features en ce moment :)