Replace base R message/warning/stop with `{cli}`

jamesmbaazam commented 3 months ago

Description

This PR closes #546.

This PR replaces the use of base condition signalling with those provided by {cli} as it offers more powerful features such as the ability to display a warning every few hours.

As part of this PR, I've updated the .lintr config file to catch future uses of base messaging. This was really useful in the refactoring exercise as it caught all occurrences. There were some false positives in the regional_epinow.R, so I embraced them in #nolint undesirable_function_linter tags to counter the configs I've set up.

Initial submission checklist

[x] My PR is based on a package issue and I have explicitly linked it.
[ ] I have tested my changes locally (using devtools::test() and devtools::check()).
[ ] I have added or updated unit tests where necessary.
[ ] I have updated the documentation if required and rebuilt docs if yes (using devtools::document()).
[x] I have followed the established coding standards (and checked using lintr::lint_package()).
[x] I have added a news item linked to this PR.

After the initial Pull Request

[ ] I have reviewed Checks for this PR and addressed any issues as far as I am able.

jamesmbaazam commented 3 months ago

Sorry, I should have marked this as a draft. Have a few simple fixes then we can try the new changes and see if happy before merging, or we can merge this and apply fancier stuff in another run. {cli} is addictive and I'm trying not to go crazy.

seabbs commented 3 months ago

Have a few simple fixes then we can try the new changes and see if happy before merging

Ha I know I think this and do fancier stuff in a new PR

epiforecasts / EpiNow2