IFAEDI / S.I

Mise en place d'un S.I avancé pour l'AEDI
3 stars 2 forks source link

Merge plus correct #31

Closed benjaminplanche closed 11 years ago

benjaminplanche commented 11 years ago

Bon, je vous préviens, c'est pas jolie comme histoire ...

En voyant la version en ligne, j'ai remarqué que la page d'accueil avait un problème avec le redimensionnement d'une image, alors que suite à la PR #15, c'était censé être bon. Après moultes recherches, le problème vient en fait d'une erreur lors du merge de la PR ajoutant le bandeau SG, ayant en partie rollbacké mes modifs.

Afin de corriger cela, j'ai voulu me mettre à jour avec IFAEDI>Master sur ma branche master perso, me retapant de la même manière l'enchainement de merges manuels ... Sauf que lors du merge fautif, j'ai gardé les bons éléments ...

Le problème, c'est que depuis, il y a eu pas mal de gros commits (restructurations) ... et apparemment ma correction sur l'ancien merge a fait prendre un autre chemin à ma branche, qui est plus considérée égale à celle du repo principal.

Conclusion, pour une simple modif' touchant 2 lignes HTML, je vous propose une PR modifiant 476 fichiers. L'avantage, c'est qu'avec ça mon repo sera à jour ... Bon courage aux autres pour en faire autant.

(Je sais pas si c'est une bonne idée ou non de merger, s'il y a une autre solution ... Au moins ca met à plat le problème)

bnjbvr commented 11 years ago

Je pense qu'il faut tenter un merge depuis une copie saine de AEDI/SI. Si personne ne s'en charge et ne met un commentaire ici pour le signaler, je m'en charge.

bnjbvr commented 11 years ago

Finalement, ça a marché sans problème en appliquant la procédure suivante:

Aucun problème, seulement 2 fichiers modifiés!