Remove access to undocumented compatibility interfaces in {mirai}

[shikokuchuo]

Prework

• Read and agree to the Contributor 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

This is mostly already done. {mirai} 0.8.3 currently has 2 compatibility fixes for crew 0.0.5 which I would like to do away with for future releases.

1. Re-creates the daemons status matrix on the fly with the previous headings. I could not find any reference to the old headers searching the code on Github so all good here.
2. 'cleanup' argument to server() still accepts TRUE/FALSE, should always be an integer value going forward. There is one remaining reference I found in a test, should just be changed to 0L:
   

   crew/tests/testthat/test-crew_worker.R

   Line 29 in ecbee58

   settings <- list(url = url, maxtasks = 1L, cleanup = FALSE)

Thanks!

[wlandau]

Thanks for the FYI. Yes, for (1), crew assumes e.g. instance instead of instance # and online instead of status_online. And for (2), 96f5d15 removes the remaining reference to a logical cleanup arg.