Cloes #1678 standardize use of `cli_inform()` and `cli_warn()` - Githubissues

Gilead-BioStats / gsm

Good Statistical Monitoring R Package

https://gilead-biostats.github.io/gsm/

Apache License 2.0

39 stars 9 forks source link

Cloes #1678 standardize use of `cli_inform()` and `cli_warn()` #1817

Closed zdz2101 closed 2 months ago

zdz2101 commented 2 months ago

Overview

cli::cli_alert_xxx() were swapped with cli_inform() of cli_warn() as "appropriate"
unit tests were modified to match the appropriate nature of the message/warning outputted

Test Notes/Sample Code

Notes:

zdz2101 commented 2 months ago

Closing as more fruitful discussion needs to be had to assign appropriate inform/warn/error