search

oceanhackweek / jupyter-image

jupyter-image
MIT License
2 stars 7 forks source link

single R script to install remotes #56

Open btupper opened 2 years ago

btupper commented 2 years ago

Script iterates through the packages identified in the YAML, returns non-zero early if an issue is encountered.

abkfenris commented 2 years ago

And need to copy remotes.yml in too.

abkfenris commented 2 years ago

Hmm, I just tried running it locally with a fake package name to see how it failed, and it just kept going instead.

btupper commented 2 years ago

Hmmm is right. I’ll have to look in am.

On Mon, Aug 15, 2022 at 10:18 PM Alex Kerney @.***> wrote:

Hmm, I just tried running it locally with a fake package name to see how it failed, and it just kept going instead.

— Reply to this email directly, view it on GitHub https://github.com/oceanhackweek/jupyter-image/pull/56#issuecomment-1216067311, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABG4SPYIVERHPHOUESIEGDTVZL26TANCNFSM56S5S4TQ . You are receiving this because you authored the thread.Message ID: @.***>

-- Ben Tupper Bigelow Laboratory for Ocean Sciences https://www.bigelow.org/ Tick forecast https://eco.bigelow.org/

btupper commented 2 years ago

OK - looks like a brainless call to library(packagename) after the install() will help throw a real error. Of course, it is meaningless if the you are trying to re-install an existing package; let's ignore that limitation!

https://stackoverflow.com/questions/26244530/how-do-i-make-install-packages-return-an-error-if-an-r-package-cannot-be-install

I'll patch it up and try again.