search

marcellinodour / MIDO-SVG

Other
0 stars 0 forks source link

Evol COMPIL #20

Closed marcellinodour closed 4 years ago

marcellinodour commented 4 years ago

PR à la bonne compilation du projet sur Master. (cf : issue #16 ) Les changements apportés ont déjà été validé à l'itération précédente.

marcellinodour commented 4 years ago

Je crains être un peu perdu sur la notion de "Séparer les tâches". Nous concevons totalement que 21 fichiers modifiés ne peut être considéré comme une tâche et doit être par conséquent séparer (code SEP). Cependant, sauf erreur de ma part, ces modifications ont été validé au PR #10 mais n'a pas fait l'objet d'un merge. Ce PR a pour tâche de restaurer la compilation du projet en réponse à l'issue #16. Que doit-on faire ?

oliviercailloux commented 4 years ago

La PR #10 ne semble pas avoir été validée, d’après ce que je vois en suivant le lien. (Sinon, vous auriez effectivement déjà pu la fusionner et il n’y aurait pas lieu d’en introduire une nouvelle qui reprend les mêmes changements.)

De toute façon, je ne crois pas qu’il faille modifier 21 fichiers pour restaurer la compilation, et en particulier, en quoi ce problème de compilation serait lié à toutes ces modifications apparaissant en PR #10.

Je vous invite donc à introduire une PR qui propose des changements minimaux dans le seul but de permettre à nouveau de compiler le projet.

marcellinodour commented 4 years ago

Pourrions-nous échanger de vive-voix sur cette PR afin de vous exposer la nécessité de ces changements ? Je ne parviens pas à suivre la procédure à suivre en prenant en compte la demande SEP.