Tim Taylor

Currently it's not clear which functions are supported/available within the package. As a concrete example, the functions `wday()` and `hour()` both work but, as they are not part of dplyr,...

Is it possible to specify column types to `df_from_csv()` and, if so, is it worth adding some examples to the documentation?

Now we're a few years past the R 3.6 release is it worth bumping the default `version` argument for `use_data()` from 2 to 3 (matching R's current default)?

`.onLoad()` seems to be setting `dplyr.show_progress` in a users options which doesn't seem necessary. https://github.com/tidyverse/dplyr/pull/7034 fixes this and maintains current logic (although AFAICT progress bars have been deprecated anyway).

.onLoad() seemed to be setting `dplyr.show_progress` in a users options which didn't seem necessary. This removes that and maintains current logic (although AFAICT progress bars have been deprecated anyway).

As part of improvements for #113 we now warn if a count column contains NA values (and nudge users to handle these before calling `incidence()`: https://github.com/reconverse/incidence2/blob/88c1fb9ecbe2ebe48973bf7815fc94451bc40e9f/R/incidence.R#L368-L375 We should provide arguments...

For vignettes, do you still advise using `html_vignette` versus `html_format`? The main difference I've observed is that `html_vignette` does not seem to allow embedding of https resources. FI - I've...

Embedded resources not being cleaned

1

Apologies if I'm misunderstanding but it doesn't seem like embedded images (not checked when other resources are embedded) are being cleaned from disk. Minimal reprex ... ``` r dir.create(dir [3]...

Allow template to be passed via `meta` argument?

4

Would it be worth allowing a template to be passed via the `meta` argument of `mark`? I've not thought too hard about this, and you likely have good reason for...