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

(R renv) Fix Renv package cache location in examples #660

Merged
merged 3 commits into from
Oct 22, 2021
Merged

(R renv) Fix Renv package cache location in examples #660

merged 3 commits into from
Oct 22, 2021

Conversation

manuelroth
Copy link
Contributor

@manuelroth manuelroth commented Oct 14, 2021

Hi all,
This PR changes the Renv package cache location in examples.md. In the latest update of renv v0.14.0 the default path to the global renv cache has changed from ~/.local/share/renv to ~/.cache/R/renv on Linux. See the renv release notes and commit for more details. Updating the cache location fixed the problem for me.

I also opened a PR on r-lib/actions to update their setup-renv action accordingly. See r-lib/actions#410 for more details. I guess more people will start running into this issue in the coming weeks. So it would be good to update the documentation on this.

Kind regards,
Manuel

@brcrista
Copy link
Contributor

It seems like there are some edge cases depending on which OS you are on and which version of R you have? From the other PR: r-lib/actions#410 (comment)

@manuelroth
Copy link
Contributor Author

Hi @brcrista, thank you for having a look at this. Yes, I have updated the PR accordingly.

@manuelroth
Copy link
Contributor Author

Hi @brcrista, I have just updated this PR with the latest changes that were just merged in r-lib/actions#414. It would be great if you could have a look at this again.

Copy link
Contributor

@brcrista brcrista left a comment

Choose a reason for hiding this comment

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

Thank you!

@brcrista brcrista merged commit 92f67a4 into actions:main Oct 22, 2021
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