search

RMI-PACTA / docker

Docker images
https://2degreesinvesting.github.io/docker/
MIT License
1 stars 0 forks source link

Do we need libgit2-dev? #28

Open maurolepore opened 3 years ago

maurolepore commented 3 years ago

usethis 2.0.0 introduces a new system dependency. I think we need to update our system/ Dockerfile.

sudo apt install libgit2-dev
cjyetman commented 3 years ago

according to renv (out <- renv::dependencies(); out$Source[out$Package == "usethis"] in the PACTA_analysis repo) usethis is only used in data-raw/data.R, which is not needed for CI/GH-Actions, and not needed for the web tool, so I would suggest removing that dependency

cjyetman commented 3 years ago

since 2DegreesInvesting/PACTA_analysis/pull/363 this may be irrelevant @maurolepore?