Réorganisation repository
publié le , mis à jourDans app/ l y a beaucoup de composants qui devrait être dans components/
Je pense que ça serait plus propre d'utiliser app uniquement pour la partie route et site web
etienneJr
dans mon souvenir, ça ne dérange pas @laem mais moi j'aime bien quand tout est bien classé, donc n'hésite pas 😁
n4n5
Okay je note
Mais le problème des réorganisation c'est que ça casse les Merges request en cours
Faudrait essayer de le faire quand il y a peu de MR active
pmiossec
@n4n5 wrote in https://codeberg.org/cartes/web/issues/1849#issuecomment-12003717:
Mais le problème des réorganisation c'est que ça casse les Merges request en cours
Pas forcement: si il y a pas beaucoup de changement dans les fichiers bougés et que donc git détecte cela comme un renommage, normalement il n'y a aucun problème, c'est comme si le fichier n'a pas bougé. A l'inverse, effectivement, c'est l'enfer...
laem
Dans app/ l y a beaucoup de composants qui devrait être dans components/
Oui carrément. Tu pourras observer que parfois je fais des déplacements, comme dans la PR actuelle catégories :)
Je suis plutôt pour qu'on le fasse route par route, doucement, à l'occasion d'une modification substantielle du code en question.
Je sais pas si vous avez vu que mon implémentation choisie de l'i18n nous a évité un déplacement de tout app/ sous "[lang]" haha