Add a ptype argument to between()

[JamesHWade]

Fixes #6906

Add ptype argument to between() function

This PR introduces a new ptype argument to the between() function, allowing users to specify the desired output type for comparisons. This enhancement is particularly useful when dealing with different data types or when specific type casting is required before comparison.

Key changes:

1. Added ptype parameter to between() function
2. Updated function logic to use ptype for casting when provided
3. Added new tests to cover ptype functionality
4. Updated documentation to explain the new argument

[krlmlr]

Nice, thanks!

[krlmlr]

These trailing spaces look odd, is this a setting in your editor?

[JamesHWade]

Not intentionally. I'm using Positron, which is still new to me.

[DavisVaughan]

I think you have to opt in to "files.trimTrailingWhitespace": true, in Positron settings, which we should probably considering automatically turning on in R files at least

I went ahead and fixed that up

[DavisVaughan]

Looking good! I first made a few tweaks of my own before leaving these comments, so make sure to git pull first before continuing your work to address these comments!

[DavisVaughan]

If you do what we suggest above, then you can revert this part

[JamesHWade]

I'm getting an error in my test when I try to revert.

[JamesHWade]

Error (test-funs.R:82:3): ptype argument works as expected with non-alphabetical ordered factors
Error in vec_recycle_common(!!!args, .size = vec_size(x)): Can't splice an object of type because it is not a vector.
Backtrace:
▆

1. ├─testthat::expect_identical(...) at test-funs.R:82:3
2. │ └─testthat::quasi_label(enquo(object), label, arg = "object")
3. │ └─rlang::eval_bare(expr, quo_get_env(quo))
4. ├─dplyr::between(x, "c", "a", ptype = x)
5. │ └─vctrs::vec_recycle_common(!!!args, .size = vec_size(x)) at dplyr/R/funs.R:55:3
6. └─rlang::abort(message = message)

[DavisVaughan]

Thanks @JamesHWade!