Standardize vec_*() error messages with check_vector_valid()

[olivroy]

Summary

This standardizes error messages for vec_*() and is inspired by the tidyverse style guide.

## before
vec_fmt_datetime(1)
#> Error in `vec_fmt_datetime()
#> ! "The `vec_fmt_datetime()` function can only be used with Date, POSIXct, or character vectors."

## now (less repetition)
vec_fmt_datetime(1)
#> Error in `vec_fmt_datetime()`
#> `x` must be a Date, POSIXct or, character vectors, not the number 1.

It gets rid of two internal functions stop_if_not_vector() and vector_class_is_valid() and consolidates them into one: check_vector_valid(), which does a mix of both.

• stops if x is not vector is valid_classes is not present
• stops if x is not vector or x doesn't inherit valid_classes

Overview of check_vector_valid()

pkgload::load_all()
#> ℹ Loading gt
# like stop_if_not_vector()
check_vector_valid(1)
# works
check_vector_valid(1, c("numeric", "integer"))
check_vector_valid(Sys.Date(), c("Date"))
check_vector_valid(Sys.Date(), "integer")
#> Error:
#> ! `x` must be integer vectors, not a <Date> object.
check_vector_valid(rep(Sys.Date(), length.out = 100), c("integer", "character"))
#> Error:
#> ! `x` must be integer or character vectors, not a <Date> object.

^{Created on 2024-05-13 with reprex v2.1.0}

Checklist

• I understand and agree to the Code of Conduct.
• I have listed any major changes in the NEWS.
• I have added testthat unit tests to tests/testthat for any new functionality.

[rich-iannone]

Great!

[rich-iannone]

Thanks again!