Use rlang::check_installed vs. rlang::is_installed more consistently?

[MichaelChirico]

I also don't have an opinion, I only note that rlang::is_installed() is already used elsewhere:

styler/R/nested-to-tree.R

Line 33 in a1038fa

#' if (rlang::is_installed("data.tree")) {

styler/R/transform-files.R

Line 182 in a1038fa

if (!rlang::is_installed("roxygen2")) {

styler/R/vertical.R

Line 24 in a1038fa

if (is_installed("prettycode")) {

However I also see check_installed:

styler/R/nested-to-tree.R

Line 50 in a1038fa

check_installed("data.tree")

styler/R/testing.R

Line 106 in a1038fa

if (write_tree) check_installed("data.tree")

So I will mark this for follow-up to use a consistent choice in the code base. (including whether to use :: as those names are in the NAMESPACE)

Originally posted by @MichaelChirico in #1267 (comment)