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

Patch 1 #845

Closed
wants to merge 2 commits into from
Closed

Patch 1 #845

wants to merge 2 commits into from

Conversation

rmflight
Copy link

Prework

Related GitHub issues and pull requests

Summary

Updates the documentation of tar_make to link to tar_invalidate to invalidate a target and show that it will cause a rebuild using tar_make so hopefully it is much more findable by users of the package.

rmflight added 2 commits May 19, 2022 09:13
Speaking as a regular user of targets, I think the tar_make should also link to and show behavior of tar_invalidate, b/c that one isn't easy to find on it's own
@wlandau
Copy link
Member

wlandau commented May 20, 2022

There are so many helpers in targets that I would prefer to handle issues like #844 at a higher level. ddf8b42 links to https://docs.ropensci.org/targets/reference/index.html#clean from tar_make() and adjusts the tar_invalidate() help file to make it more obvious that it causes a target to rerun.

I'm not sure if I can recommend tar_invalidate() as a way to rerun a target in all use cases. For example, if the metadata with runtime etc. is more important in the moment than the actual return value, then I would recommend tar_delete(). And tar_cue(mode = "always") works without deleting data.

@wlandau wlandau closed this May 20, 2022
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