[Bioc-devel] Aliasing `]` breaks BiocCheck::BiocCheck() version 1.18.0

[Shepherd, Lori]

Can you please open an issue for this so we don't lose track of it -

https://github.com/Bioconductor/BiocCheck/issues



Lori Shepherd

Bioconductor Core Team

Roswell Park Cancer Institute

Department of Biostatistics & Bioinformatics

Elm & Carlton Streets

Buffalo, New York 14263

________________________________
From: Bioc-devel <bioc-devel-bounces using r-project.org> on behalf of Shian Su <su.s using wehi.edu.au>
Sent: Monday, December 17, 2018 8:34:10 PM
To: bioc-devel
Subject: [Bioc-devel] Aliasing `]` breaks BiocCheck::BiocCheck() version 1.18.0

Hi all,

If you put

foo <- `[`

Somewhere in a package, it will trigger

Error in makeUsageCollector(fun, ...) : only works for closures

In BiocCheck::BiocCheck() (version 1.18.0). This comes from

if (typeof(fun) != "closure")
        stop("only works for closures")

In codetools::makeUsageCollector(), but

> typeof(`[`)
## "special"

Not that it matters for my use-case because I had discovered magrittr�s extract alias, but it might be an edge case worth covering, especially since the error message is so cryptic.

Kind regards,
Shian Su

_______________________________________________

The information in this email is confidential and intend...{{dropped:29}}