How should we structure CICD checks

[epijim]

For the CICD checks/protections, I was thinking the following:

• to be able to push code to master, it must pass R-CMD-CHECK, the covr code coverage must run (this should happen if the prior worked), and the automated pkgdown docs build and deployment must run (again should be covered by R-CMD-CHECK unless there is funky code in the README.Rmd or online only vignettes)
• for develop - all the checks above run, but it doesn't stop you merging into develop if one fails (alternative would be to also 'protect' develop, and require working code before you can merge in).

What are your thoughts on how to handle develop? Do we let people work in potentially broken code then tidy it on develop, or do we make them fix it before a merge in to develop?

[epijim]

changes being implemented in #101