Parties communes - Githubissues

jengelaere commented 2 years ago

Coucou JD, Un petite demande de MR, à partir d'une branche qui n'est peut être pas complètement ok. Je patauge un peu pour mettre à jour le readme.md de ce repo :

si je le pousse, les actions github ne sont pas contentes car elles ne veulent pas écraser les changements poussés, et l'étape du render-README échoue ;
si je ne le pousse pas, la Ci tourne correctement mais ne copie pas le readme.md dans la branche, c'est peut être pcq on est pas dans master, mais je n'ai pas osé tester directement dans master... du coup 🥺 peux-tu y jeter un oeil ? Dans cette branche :
mise à jour des noms des ministères,
ajout d'un dossier parties_communes avec des blocs à intégrer dans les différents support, pour une mise à jour centralisée (c'est l'idée que je vais tester dans le m1...)
ajout des liens vers les environnements SSP cloud de chaque repo dans la page d'accueil docs/index.html (issue https://github.com/MTES-MCT/parcours-r/issues/52)
mise à jour des auteurs dans index.html

jdlom commented 2 years ago

👍 je regarde

jdlom commented 2 years ago

si je le pousse, les actions github ne sont pas contentes car elles ne veulent pas écraser les changements poussés, et l'étape du render-README échoue ;

si je ne le pousse pas, la Ci tourne correctement mais ne copie pas le readme.md dans la branche, c'est peut être pcq on est pas dans master, mais je n'ai pas osé tester directement dans master...

En effet, le github_action ne fonctionnera que sur la branche master mais je voulais quand même savoir si le render du Readme.Rmd passait.

jdlom commented 2 years ago

@jengelaere tu penses qu'on pourra merger quand ?

jengelaere commented 2 years ago

coucou @jdlom quand tu veux, c'est prêt. Juste préviens moi avant de supprimer la branche, que j'adapte les liens du support M1 à la branche master

jengelaere commented 2 years ago

Mes réserves initiales étaient sur le readme.md mais tu m'a rassurée, tout est ok

jdlom commented 2 years ago

vamos je merge, je pense que ca va marcher, et au pire je regarderai si ca marche pas :)

jdlom commented 2 years ago

@jengelaere, bon et bien ca a foire.

jdlom commented 2 years ago

Le build passe mais il n'y a pas l'autocommit du README.md mais je viens de voir que tu l'avais mis au .gitignore donc c'est probablement pour ca.

jengelaere commented 2 years ago

ahh la vilaine ! 🙊 Désolée, je pensais que c'était une bonne idée, pour éviter de le pousser et de mettre en échec le pipeline

jdlom commented 2 years ago

non mais c'est normal en fait c'est une bidouille que j'ai faite !

jdlom commented 2 years ago

@jengelaere j'ai fait une PR : https://github.com/MTES-MCT/parcours_r_socle_introduction/pull/33

MTES-MCT / parcours-r

Parties communes #54