Allow exceptions on use of <<-

[mpadge]

As explained by @bnicenboim in this submission comment ropensci/software-review#573 (comment), the memoise package explicitly recommends assigning all memoised versions of functions to the global namespace, within an .onLoad function. This use could be detected and allowed, instead of current failure for any and all uses of <<-.

[mpadge]

@bnicenboim The above commits now permits use of <<- when used in .onLoad functions for memoise. Your package still fails because it also uses <<- reticulate::import in .onLoad. This is also now trivial to permit, but I'd first like to think about more general implications of permitting such more general usage. I expect to return to this issue within the week. Thanks

[bnicenboim]

ok, thanks. I was following the recommendation in "Using reticulate in an R Package" vignette .

[mpadge]

Re-opening to modify current code to allow any usage of <<- in .onLoad() - and only there! - and to issue a 👀 message instead of ✖️