Closed richfitz closed 1 year ago
Patch coverage: 100.00
% and project coverage change: +0.05
:tada:
Comparison is base (
91aa1f5
) 99.89% compared to head (c9677f8
) 99.95%.:exclamation: Current head c9677f8 differs from pull request most recent head 0a4e41e. Consider uploading reports for the commit 0a4e41e to get more accurate results
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
Merge after #93, it contains those commits
This PR moves to use callr for worker spawning, which I think makes a few things more robust, particularly the way that callr arranges for child R sessions to be arranged similarly to the parent ones (library paths, etc). We already use callr for running tasks in a separate processes.
rrq_worker_spawn
is now a little R6 class that can be poked at; there's a bit of simplification through the tests and vignettes due to thisWith this change in we barely use docopt, so I might look over that interface next