-
Notifications
You must be signed in to change notification settings - Fork 76
Description
Prework
- Read and agree to the code of conduct and contributing guidelines.
- If there is already a relevant issue, whether open or closed, comment on the existing thread instead of posting a new issue.
- For any problems you identify, post a minimal reproducible example like this one so the maintainer can troubleshoot. A reproducible example is:
- Runnable: post enough R code and data so any onlooker can create the error on their own computer.
- Minimal: reduce runtime wherever possible and remove complicated details that are irrelevant to the issue at hand.
- Readable: format your code according to the tidyverse style guide.
Description
Hey @wlandau,
I think the functions tar_target()
and tar_option_set()
are missing documentation for error = "workspace"
. Right now the documentation makes it seem like the only options are "stop"
and "continue"
. I vaguely recall this used to be documented but seems to be absent now.
Character of length 1, what to do if the target runs into an error. If "stop", the whole pipeline stops and throws an error. If "continue", the error is recorded, but the pipeline keeps going.
While I'm thinking about it, should error = "workspace"
even be an option? Shouldn't a user have the option to save a workspace regardless of whether the pipeline stops or continues when an error is thrown? In other words, I would think error
should only be able to take the values "stop"
or "continue"
, and in either case, if the failed target is named in workspaces = "..."
, then the workspace is saved. Thoughts? It's late and I'm tired, so maybe this won't make sense when I look at it again in the morning.