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

[Tierney, Luke]

Codetools should probably be ignoring those. Will have a look

Sent from my iPhone

> On Dec 18, 2018, at 6:54 AM, Shepherd, Lori <Lori.Shepherd using RoswellPark.org> wrote:
> 
> 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}}
> 
> _______________________________________________
> Bioc-devel using r-project.org mailing list
> https://stat.ethz.ch/mailman/listinfo/bioc-devel