Compatibility with dev version of purrr

[larmarange]

See tidyverse/purrr#969

• Remove Remotes field in DESCRIPTION before merging

[codecov]

Codecov Report

Merging #181 (a32e76f) into main (3cdccea) will increase coverage by 1.35%.
The diff coverage is 100.00%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #181      +/-   ##
==========================================
+ Coverage   97.34%   98.70%   +1.35%     
==========================================
  Files          38       38              
  Lines        1694     1695       +1     
==========================================
+ Hits         1649     1673      +24     
+ Misses         45       22      -23     

Impacted Files	Coverage Δ
R/assert_package.R	94.02% <100.00%> (+34.93%)	⬆️

[larmarange]

The checks failed with R 3.6 on Windows but it could be due to the fact that the remotes version of purrr is not available.

We need to re-check once the Remotes filed removed from DESCRIPTION.

[ddsjoberg]

looks great! once this is merged, i'll work on gtsummary too

[larmarange]

Thanks. Let me remove the Remotes from the DESCRIPTION, re-run all checks and then merge.

We may submit a new version on CRAN just after, what do you think?

[ddsjoberg]

We may submit a new version on CRAN just after, what do you think?

I looked in the purrr code and it seems to be a soft deprecation, meaning that warnings only appear in dev environments. So I don't there is any rush to release before their purrr 1.0 release

[larmarange]

It could be a problem for purrr team as CRAN will run a revdep check.