document that main arguments are captured as quosures

[simonpcouch]

Closes #662.🦀

If we think the model_type() pseudocode may be confusing for folks, we could make this into a proper template that could take the name of the model type function as an input. :)

[hfrick]

Hm, I think we should change something about that model_type() snippet. It using the actual modelling function would be nice. A possibly easier solution is to explicitly say that this is pseudo-code. Either solution is fine with me.

[hfrick]

I think the pattern is "every arg apart from mode and engine" but I haven't checked all of them. I did hit bag_tree() though which has non-NULL defaults for some main args and all of them get captured as quosures:

parsnip/R/bag_tree.R

Lines 25 to 37 in 332de18

	bag_tree <-
	function(mode = "unknown",
	cost_complexity = 0,
	tree_depth = NULL,
	min_n = 2,
	class_cost = NULL,
	engine = "rpart") {
	args <- list(
	cost_complexity = enquo(cost_complexity),
	tree_depth = enquo(tree_depth),
	min_n = enquo(min_n),
	class_cost = enquo(class_cost)
	)

[simonpcouch]

Ah, good point. Thank you!

[simonpcouch]

I'm with ya. Let's make this into a proper template and substitute the function name in.

[hfrick]

That's so neat! I didn't know about that 👀 🤩

[github-actions]

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.