Warning with `%||% reexport from rlang

[lionel-]

I'm seeing this report in CRAN checks:

Package: scCustomize
Check: whether package can be installed
New result: WARNING
  Found the following significant warnings:
    Warning: replacing previous import ‘Seurat::%||%’ by ‘rlang::%||%’ when loading ‘scCustomize’

Currently rlang::%||% is copied into the SeuratObject namespace:

`%||%` <- rlang::`%||%`

This should be a reexport instead, like you did in Seurat with

#' @importFrom SeuratObject %||%
#' @rdname reexports
#' @export
#'
SeuratObject::`%||%`

Hopefully that will be enough to fix the warning in scCustomize. If not, you might need to reexport %||% from Seurat directly from rlang instead of SeuratObject.

[mojaveazure]

Hi Lionel,

Thanks for reporting. We did it this way to control how %||% shows up in documentation (roxygen2 is rather aggressive about how it documents reexports). We'll adjust this to properly reexport rather than redefine %||%

[lionel-]

Thanks!