don't use glue::glue() in recipe step #5
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hello @ashbythorpe 👋
I was doing some work in recipes (tidymodels/recipes#1199) and
step_nest()
came up in some testing. The step producesglue
objects which recipes doesn't know how to handle:https://github.com/tidymodels/recipes/blob/914cdd443d49b77e22d2b651cb89aa23d0606e8a/R/misc.R#L158-L161
This PR changes the step so a factor is returned instead.
If you are able to pass this change along in a update on CRAN that would be very helpful. I'm planning to do a recipes release in about a month or so.