skip check_cols in gennerate when type = 'bootstrap'

[liubao210]

When type = "bootstrap", the variable parameter is not required? According to the existing code, executing this statement will report an error generate(mtcars, reps = 10, type = 'bootstrap') So I made changes to skip check_cols when type = "bootstrap"

[simonpcouch]

Thanks for the PR! Can you please confirm that this PR doesn't duplicate #425 and, if not, provide a minimal reprex (reproducible example) demonstrating the issue that your PR addresses and how the package's behavior will change?

[liubao210]

Thanks for the PR! Can you please confirm that this PR doesn't duplicate #425 and, if not, provide a minimal reprex (reproducible example) demonstrating the issue that your PR addresses and how the package's behavior will change?

Before this change I execute the code mtcars |> generate(reps = reps, type = "bootstrap")，got "error: The variables argument should be one or more unquoted variable names (not strings in quotation marks)."
because i have to

Thanks for the PR! Can you please confirm that this PR doesn't duplicate #425 and, if not, provide a minimal reprex (reproducible example) demonstrating the issue that your PR addresses and how the package's behavior will change?

Before this change I execute the code mtcars |> generate(reps = reps, type = "bootstrap"), got "error: The variables argument should be one or more unquoted variable names (not strings in quotation marks)."
I have to assign a value to variables mtcars |> generate(reps = reps, type = "bootstrap", variables = cyl), which is useless? so i made this change,Thanks!

[simonpcouch]

Ah, I see. I think skipping check_cols entirely is a bit too blunt of an approach, but have opened an issue #448 for more discussion. Thanks again for the PR. :)

[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.