Marcin
Marcin
@llrs-roche I think this PR is outdated and there are merge conflicts. Also, it is not enough to accept community PR. You need to merge it on your own as...
I think you should prepare a file under which we could comment
@gogonzo > No hardcoded argnames in the module wrapper I think those comments are outdated (they have a date of 3 weeks, even though they appeared with a review that...
Making it a regular PR so we can get CI/CD running
Thanks @averissimo for the thorough review! Those are all really useful updates to the current flow :)!
@averissimo the bug that you reported I was aware of > Found a 🐞 bug: Can only add 1 card > How to reproduce: > Load example shiny app >...
@kumamiao yes, it's related to tmg. The table needs to be added here https://github.com/insightsengineering/teal.modules.general/blob/main/R/tm_variable_browser.R#L514
> There is https://github.com/insightsengineering/teal/issues/1404#issuecomment-2650872198 about having a git hook locally. usethis::use_readme_rmd() generates a hook to be added to .git/hooks/pre-commit (not sure if this could be shared via .pre-commit-config.yaml). Our .pre-commit-confit.yaml...
@llrs-roche > From pkgdown::build_home(): >> If you use index.Rmd or README.Rmd it's your responsibility to knit the document to create the corresponding .md. pkgdown does not do this for you...
> If we want to include we would need to render it into different format but not sure how it would play with the CI and the pkgdown internal machinery....