tidymodels/recipes

add helper function for `try(rlang::eval_tidy(cl), silent = TRUE)`

EmilHvitfeldt opened this issue · 2 comments

EmilHvitfeldt commented 
          I think it will be worth it to have a helper function here. Using the above code for helping. There are many places in {recipes} where we `try(rlang::eval_tidy(cl), silent = TRUE)` and it smells a little like we need a helper function

Originally posted by @EmilHvitfeldt in #1383 (comment)

simonpcouch commented

Another bit that feels worth mentioning is that many usages of this pattern in this package call as.character() on the condition object rather than just passing the condition to the parent argument.

  • 👀1