Skip to content

Add preview script#591

Open
nicotlm wants to merge 20 commits into
InseeFrLab:mainfrom
nicotlm:dev_preview
Open

Add preview script#591
nicotlm wants to merge 20 commits into
InseeFrLab:mainfrom
nicotlm:dev_preview

Conversation

@nicotlm

@nicotlm nicotlm commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Close #588

Update du GHA workflow de preview :

  • passage à un container Onyxia pour gagner du temps sur la préview (plus R à installer)
  • on n'installe plus dependancies.sh (fonctionne tb sans => inutile ? )
  • mise à jour quand le workflow démarre : PR faite sur main, ouverte, nouveau push, ou fermée.
  • ajout remove preview quand la PR est fermée

Checklist:

En faisant cette pull request, je confirme que :

  • J'ai lu le guide des contributeurs
  • Ma proposition respecte les canons formels de la documentation utilitR
  • Les exemples de code R ont été testés sur ma machine
  • J'ai testé, sur ma machine, que la documentation compile avec mes ajouts (quarto::quarto_preview()) produit un résultat
  • Si j'y suis invité (cela ne fonctionne pas pour toutes les pull requests), je consulte le site de prévisualisation https://${BRANCH_NAME}--preview-docr.netlify.app/

@nicotlm nicotlm requested a review from JulienBlasco June 18, 2026 09:53
@linogaliana

Copy link
Copy Markdown
Contributor

Hello ! Merci de la suggestion, ça a l'air cool ! Tu as repris ce que tu as fait avec le SSPHub ?

@JulienBlasco JulienBlasco left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci Nicolas ! Je ne peux pas vraiment valider le code ni m'assurer que ça fonctionnera, mais l'approche me paraît bonne. Je laisse Lino merger si ça lui plaît :)

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 this pull request may close these issues.

Corriger la prévisualisation des PR

3 participants