search

ddotta / tablexlsx

R package to export data frames from R to xlsx workbook
https://ddotta.github.io/tablexlsx/
Other
16 stars 1 forks source link

Fix CI bugs caused by evolution-api #31

Closed JulienBlasco closed 5 months ago

ddotta commented 6 months ago

Merci @JulienBlasco en espérant que ces modifications corrigent l'intégration continue...

Je vois que ta PR est encore au statut de draft, pour quelle raison ?

Comme tu as modifié, le nom de la vignette, je pensais à modifier aussi ici :

https://github.com/ddotta/tablexlsx/blob/5e12edbfc9a1553df71f603afa461e545ed539c5/README.md?plain=1#L64

Là :

https://github.com/ddotta/tablexlsx/blob/5e12edbfc9a1553df71f603afa461e545ed539c5/R/toxlsx.R#L8

Et là :

https://github.com/ddotta/tablexlsx/blob/5e12edbfc9a1553df71f603afa461e545ed539c5/R/toxlsx.R#L34

JulienBlasco commented 6 months ago

Hello Damien, j'ai initié la pull request parce que ça me permet de relancer toute la CI à chaque commit. Mais elle est encore en draft parce qu'elle n'est pas du tout finie ! J'ai corrigé une erreur de pkgdown mais je n'ai pas encore réussi à comprendre pourquoi les tests plantent sur le CI.

Je passerai la pull request en "ready for review" une fois que les tests passeront.

JulienBlasco commented 5 months ago

Ça semble fonctionner ! J'ai changé la valeur par défaut de ByGroup et bygroup dans les deux fonctions. Je n'ai toujours pas réussi à reproduire le comportement de la CI (ça marchait toujours chez moi) donc c'était assez difficile à débuguer.

ddotta commented 5 months ago

🎉 🎉 🎉 ça a marché !
Trop bien même si j'avoue que je ne comprends pas pourquoi il n'aimait pas les NULL

JulienBlasco commented 5 months ago

Super, de toutes manières je pense que ce sont des valeurs par défaut plus cohérentes que du NULL. resolves #30 ✔️

Il reste peut-être à documenter ces nouvelles fonctionnalités dans les vignettes d'exemple.