Replace base messages with rlang and cli

[galachad]

• Replace base message with rlang
• Move vignette to package space
• Adds missing metadata in vignette

Close #78

• Place Closes #<insert_issue_number> into the beginning of your Pull z
  Request Title (Use Edit button in top-right if you need to update)
• Code is formatted according to the
  tidyverse style guide. Run
  styler::style_file() to style R and Rmd files
• Updated relevant unit tests or have written new unit tests, which should
  consider realistic data scenarios and edge cases, e.g. empty datasets, errors,
  boundary cases etc. - See
  Unit Test Guide
• If you removed/replaced any function and/or function parameters, did you
  fully follow the
  deprecation guidance?
• Update to all relevant roxygen headers and examples, including keywords
  and families. Refer to the
  categorization of functions to tag appropriate keyword/family.
• Run devtools::document() so all .Rd files in the man folder and the
  NAMESPACE file in the project root are updated appropriately
• Address any updates needed for vignettes and/or templates
• Update NEWS.md if the changes pertain to a user-facing function (i.e. it
  has an @export tag) or documentation aimed at users (rather than developers)
• Build oak site pkgdown::build_site() and check that all affected
  examples are displayed correctly and that all new functions occur on the "Reference" page.
• Address or fix all lintr warnings and errors - lintr::lint_package()
• Run R CMD check locally and address all errors and warnings - devtools::check()
• Link the issue in the Development Section on the right hand side.
• Address all merge conflicts and resolve appropriately
• Pat yourself on the back for a job well done! Much love to your accomplishment!

[github-actions]

Package	Line Rate	Health
sdtm.oak	93%	✔
Summary	93% (953 / 1021)	✔

[ramiromagno]

Hi @galachad:

• Can't we stick with cli::cli_abort/warn/inform() all across instead of rlang::abort/warn/inform()? Just for consistency.
• The S3 registration of mutate.cnd_df is not a mistake by you (it's a commit by me made earlier to this PR, see d6e3d92. So think commit d0873ac should be reverted.
• Locally, I still get the testthat snapshot blfl related issues when I run devtools::check() on this PR, so I am guessing that the CI/CD checking this PR is not using the latest admiraldev version, is it?

[galachad]

Hi @ramiromagno

The snapshot issue is fixed #73 or #75 we do not yet decide whether we are merged.

The different in the message was introduced in admiraldev v1.1.0 package.

For export of mutate.cnd_df: thanks was pointing that out, I was thinking that I exported that by the mistake.

Let's use cli everywhere for the errors and messages.

[ramiromagno]

All good.