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

feature request for crancache_remove #43

Open
bbolker opened this issue May 10, 2021 · 1 comment
Open

feature request for crancache_remove #43

bbolker opened this issue May 10, 2021 · 1 comment

Comments

@bbolker
Copy link

bbolker commented May 10, 2021

I have two minor usability gripes with crancache_remove():

  • the regex-matching feature is a little dangerous; I can't remember when, but I think I've accidentally removed packages this way in the past
  • I have a hard time remembering that it's not remove_packages() (in parallel with update_packages(), install_packages()

Adding a remove_packages() function that did string-matching rather than regex-matching would preserve backward compatibility ...

@gaborcsardi
Copy link
Member

If we had a remove_packages() that should remove the packages from the library and not from the cache. At least this is how the other replacement functions work.

We could have crancache_remove(..., fixed = TRUE) or something similar? A PR is welcome for this. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants