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

Checklist and renv updates #5

Merged
merged 7 commits into from
Jan 22, 2024
Merged

Checklist and renv updates #5

merged 7 commits into from
Jan 22, 2024

Conversation

EmmaCartuyvels1
Copy link
Collaborator

The version of R and all packages were updated and changed in the renv files.
All necessary checklist issues were adressed.
Targets pipeline was rerun to ensure everything worked.

Copy link
Collaborator

@hansvancalster hansvancalster left a comment

Choose a reason for hiding this comment

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

Dat ziet er goed uit. De online (continuous integration) check faalt nog, maar dat is dus omwille van het probleem met check_lintr en de renv folder. Ik zag nog wel enkele notes in de online check die je misschien nog eens moet bekijken. Je kan dit bekijken door op details te klikken bij de check, maar ik zal al approven.

@hansvancalster hansvancalster self-requested a review January 19, 2024 13:52
Copy link
Collaborator

@hansvancalster hansvancalster left a comment

Choose a reason for hiding this comment

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

Ik zie nu pas dat in plaats van renv te update in het source/targets/mas_steekproef_zn_lm/mas_steekproef_zn_lm.Rproj er een nieuw renv package management werd toegevoegd aan het Rstudio project in de root van de repo (dus aan ./mbag-mas.Rproj). Dat was niet de bedoeling denk ik. Kan je deze wijzigingen ongedaan maken en enkel de renv van targets project updaten? Laat maar weten als je hulp nodig hebt.

@hansvancalster
Copy link
Collaborator

Kan je deze wijzigingen ongedaan maken en enkel de renv van targets project updaten? Laat maar weten als je hulp nodig hebt.

Er is een functie die dit doet: https://rstudio.github.io/renv/reference/activate.html. Dus het mbag-mas.Rproj opstarten en dan renv::deactivate(project = NULL, clean = TRUE) uitvoeren zal renv verwijderen van dat RStudio project.
De verwijderde bestanden kan je dan committen en pushen.
Daarna kan je het Rstudio project starten met de targets pipeline, waar de renv.lock file nog de oude R versie heeft. Die kan je updaten door renv::snapshot() uit te voeren.

@EmmaCartuyvels1 EmmaCartuyvels1 merged commit e1d4732 into main Jan 22, 2024
1 check failed
@EmmaCartuyvels1 EmmaCartuyvels1 deleted the checklist_updates branch January 22, 2024 09:19
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.

2 participants