transition remaining rlang warnings and errors to cli

tidymodels / parsnip

A tidy unified interface to models

https://parsnip.tidymodels.org

Other

601 stars 89 forks source link

transition remaining rlang warnings and errors to cli #1162

Closed simonpcouch closed 3 months ago

simonpcouch commented 3 months ago

Closes #1081. (!!!)

Definitely many opportunities to write out some of these errors with rlang type checkers, but... one step at a time. :)

This will definitely cause some snap changes at least in extratests and likely in parsnip extensions. I say we update extratests snaps along with this PR but wait to make PRs to parsnip extensions until we've followed this PR up with another in parsnip that integrates rlang type checkers.

topepo commented 3 months ago

I'm working on a corresponding extratests PR to update snapshots

github-actions[bot] commented 2 months ago

This pull request has been automatically locked. If you believe you have found a related problem, please file a new issue (with a reprex: https://reprex.tidyverse.org) and link to this issue.