Gábor Csárdi

Cool. `declare` is fine in magrittr, I guess, people can copy it if they want to. What I am not sure about, however, where the `magrittr::declare( .

@jennybc Then just put this in the package and the NOTE goes away. You can change the exact text according to your mood. :)

That just seems more cumbersome: :) ``` r if(getRversion() >= "2.15.1") utils::globalVariables(c(".")) ``` More seriously, it is indeed better, it just instructs the check tool to ignore the dot, instead...

Sometimes you have to use sigkill, because R in the subprocess is not interruptible.

@richfitz You can add some `$signal()` calls to your event loop layer if you want a graceful shutdown. processx is a bit too low level to implement this, without a...

As an alternative that is generic, how about a marker that makes covr descent deeper? E.g. the list / env could have a `covr_this = TRUE` attribute.

Yeah, that does sound good, but it might be tricky to implement. E.g. in this case where would you put the comment? ```r api

@brodieG Maybe not, but it might be too late to change this now. But either a global or local opt-in might make sense. Or both. We can also make a...

So how about having a global opt-in full traversal for a start? Not entirely trivial, but probably not too bad to implement, e.g. I think debugme already has a full...

I would look at the `path_*()` functions: ``` ❯ fs::path_ fs::path_abs fs::path_ext_set fs::path_package fs::path_common fs::path_file fs::path_real fs::path_dir fs::path_filter fs::path_rel fs::path_expand fs::path_has_parent fs::path_sanitize fs::path_expand_r fs::path_home fs::path_split fs::path_ext fs::path_home_r fs::path_temp fs::path_ext