Behaviour of load_code in roxygenize()

[jennybc]

This devtools discussion around updating devtools::document() in light of roxygen v6.1.0 needs to continue here:

r-lib/devtools#1849

Excerpt to remind what this is about:

... the helpers = FALSE behaviour (in the sense of load_all(pkg$path, helpers = FALSE)) has been lost. Is that worth taking a stand on?

Two additional points raised by @jimhester in the other thread:

Maybe we should be calling pkgload::load_all(helpers = FALSE) in roxygen2 and maybe export_all = FALSE to more closely mimic a normal package namespace? I am not sure what roxygen2 assumes in regards to exports.

Also potentially recompile = FALSE, which would be a way to fix #771, but maybe would cause other issues if the dll was not built.

[hadley]

Unfortunately there's no way to completely suppress compilation in pkgload::load_all() — recompile = FALSE is already the default, but what we actually need is compile = FALSE. I'll implement that in pkgload and then once that's on CRAN (hopefully later this week), I'll do pkgload::load_all(compile = FALSE, helpers = FALSE), which should resolve the worst of the problems related to this change.

[hadley]

Should be better now. Can someone else confirm that this fixes the problem before I push to CRAN?

[hadley]

I need to release today to fix issues with CRAN R cmd check and I'll hope silence indicates no major problems.