local_tempfile() fails to close the connection when used in examples with lines=

[MichaelChirico]

local_tempfile() fails to close the connection when used in examples with lines=

[lionel-]

See the NEWS file, you need to actively set the withr.hook_source option when sourcing outside of the global env.

I would just use local() in that example though.

[MichaelChirico]

I would rather not clutter up package examples with extra nesting/option setting... let me take a look tomorrow at how else to support this use case. We're not failing on CRAN so presumably it's an issue with the CI script.

FWIW I would call out this use case (local_tempfile() inside package examples) in the NEWS, I read that bullet but didn't make the connection because the source() usage is several layers deep.

[lionel-]

CRAN is sourcing in the globalenv, which explains why source() works out of the box.

By the way @gaborcsardi let me know that this only works because your example is conditional. If you remove the condition, the generated R code will no longer be fenced by source(), and withr will stop working 100% as expected. The exit handlers will be run after all examples have completed rather than at the end of the particular example withr was used. So local() is definitely still recommended in examples.

[MichaelChirico]

Got it, thanks for the elaboration. In that case I will just stick with un-hooked file cleanup, using local() in examples will be confusing for novice readers IMO.

I see two other CRAN packages that may be affected:

https://github.com/search?q=org%3Acran+path%3A.Rd+%2Fwithr%3A%3Alocal_tempfile%5B%28%5D%2F+-repo%3Acran%2Flintr&type=code