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

Use secure CRAN mirror #202

Merged
merged 2 commits into from
Feb 5, 2021
Merged

Use secure CRAN mirror #202

merged 2 commits into from
Feb 5, 2021

Conversation

csgillespie
Copy link
Contributor

A simple one character PR to use the https CRAN mirror instead of the http

@sckott sckott added this to the v1.0 milestone Feb 4, 2021
@sckott
Copy link
Collaborator

sckott commented Feb 4, 2021

thanks!

@sckott
Copy link
Collaborator

sckott commented Feb 4, 2021

Actually, @csgillespie we should probably switch to using https://cloud.r-project.org/ - i think that's the global CDN they have and is default for their IDE. Can you make that change?

@csgillespie
Copy link
Contributor Author

Good spot - List of cran mirrors. Sorry, I didn't think to check.

@csgillespie
Copy link
Contributor Author

As an aside, a reasonable additional argument to debian_install_r(), would be rprofile with the default value of options(repos=c(\'https://cloud.r-project.org/\')).

The rationale is that some people might want to use the RSPM, but you need to be careful with your Ubuntu version. You might also want to tweak other arguments, e.g. Ncpus in options

Happy to make another PR if you want.

@sckott sckott merged commit e719618 into pachadotdev:master Feb 5, 2021
@sckott
Copy link
Collaborator

sckott commented Feb 5, 2021

Good idea to have a parameter rprofile - can you send another PR for that?

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