Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mettre à jour la CI pour que tous les supports soient rendus #36

Closed
jengelaere opened this issue Oct 18, 2021 · 4 comments · Fixed by #46
Closed

Mettre à jour la CI pour que tous les supports soient rendus #36

jengelaere opened this issue Oct 18, 2021 · 4 comments · Fixed by #46
Assignees

Comments

@jengelaere
Copy link
Collaborator

No description provided.

@jengelaere jengelaere changed the title Mettre à jour la CI pour que tous les supportss Mettre à jour la CI pour que tous les supports soient rendus Oct 18, 2021
@fuguwoofer
Copy link

Bonjour les collègues, @jengelaere, @jdlom,

avant même d'attaquer le sujet de l'intégration continue, je vous propose une étape intermédiaire à consulter dans la branche restart-effort-2022-09.

Je viens de finaliser ce qui m'était une étape d'appropriation personnelle indispensable, c'est-à-dire : une refactorisation du code pour piloter plus simplement la reconstruction des différents éléments qui entrent dans la composition de l'ensemble du matériel de formation tel qu'il était prévu. Ces modifications lourdes peuvent dérouter les gradients du temple.

Cependant, il faudra prendre le temps de bien lire les logs de modification que j'ai tenu à empiler de manière didactique. La simplification pourra étonner mais le lecteur rigoureux constatera qu'aucune ligne de code 'fonctionnelle' n'a disparue.

Merci de me dire ce que vous en pensez, je sais que ce ne sera pas simple, mais comme j'entends prendre temporairement le leadership de la démarche, j'espère que ce sera l'occasion pour vous de me déléguer un peu de confiance. Donc si vous n'avez pas le temps de rejouer les commits (il y a près de 20 heures de travail sous vos yeux), vous pouvez toutefois constater l'état fonctionnel, en :

  • switchant sur la branche restart-effort-2022-09
  • en exécutant ligne à ligne le script env-setup.R qui permet d'installer les pré-requis nécessaires (personnellement j'utilise renv, si vous pas : veillez à commenter la première commande)
  • exécuter le script : build_utils.R qui va recréer dans un répertoire build l'ensemble du site tel qu'il avait été assemblé par Maël et Marouane.

Cordialement,
Laurent

@MaelTheuliere
Copy link
Contributor

Hello les anciens collègues, un petit passage ici pour saluer ton travail de nettoyage Laurent, c'est je trouve bien plus lisible. Juste une remarque, ce qui complexifie la CI, c'est que qu'avec Rmarkdown, on doit compiler séparément les slides le book puis le site, avec quarto un quarto::quarto_render() va faire tout à la fois.

@fuguwoofer
Copy link

@MaelTheuliere Ton retour me touche et ton passage fait plaisir !

D'autant que tu me fais découvrir une nouvelle lib. Je dois toutefois avouer que je n'ai jamais eu recours à l'intégration continue (je n'en connais que grossièrement la définition) donc je ne sais pas mesurer la complexité de sa mise en pratique et encore moins dans l'espace github.

Mais, pour moi en mode dev, à coups de RScript build_utils.R je régénère tout d'un coup, je n'ai donc pas cherché plus loin (flemme?). J'ai par contre constaté un bug étrange en sourçant au travers de RStudio ... mais c'est une autre histoire.

Allons découvrir quatro ... ce que c'est que la curiosité tout de même :-)
@+

@MaelTheuliere
Copy link
Contributor

Les fonctions que tu as faites peuvent alléger la partie ci qui reproduisait le même processus par étape de production des slides, du book, puis du site. Il manque la partie construction du fichier image de la page de garde qui est ensuite utilisée sur le site en vignette et des pdf des slides qui est aussi constitué dans la ci et pourrait être déporté dans une fonction. Cf le yaml actuel pour voir le code utilisé à cette fin : https://github.com/MTES-MCT/parcours_r_module_publication_rmarkdown/blob/dev/.github/workflows/distill-dev.yaml
Tu vas avoir la grande joie à découvrir l'intégration continue, un monde nouveau va s'ouvrir à toi ;-)

@jengelaere jengelaere assigned jengelaere and unassigned jdlom Jan 10, 2025
jengelaere added a commit that referenced this issue Jan 10, 2025
@jengelaere jengelaere linked a pull request Jan 10, 2025 that will close this issue
jengelaere added a commit that referenced this issue Jan 10, 2025
 la classique : il manquait COGiter dans les dépendances pour créer la carte des référents R
jengelaere added a commit that referenced this issue Jan 10, 2025
 typo , en trop
jengelaere added a commit that referenced this issue Jan 10, 2025
jengelaere added a commit that referenced this issue Jan 10, 2025
une ligne en trop dans DESCRIOPTION pour que les packages github s'intallent ?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants