Pkgdown is failing about skimr

[michaelquinn32]

Fixed with the release of 2.1.4

[maelle]

👋 @elinw (and @michaelquinn32)!

Interesting 😅 I see r-lib/pkgdown#677

Does the one_skim_df have a print method? I'll try to write a reprex for downlit. 🤔

[maelle]

A reprex would be this code

code <- "# Compare
iris %>%
  skim() %>%
  dplyr::select(n_missing)

iris %>%
  skim() %>%
  focus(n_missing)

# This is equivalent to
iris %>%
  skim() %>%
  dplyr::select(skim_variable, skim_type, n_missing)"

downlit::evaluate_and_highlight(code)

[maelle]

code <- "
library(skimr)
# Compare
iris %>%
  skim() %>%
  dplyr::select(n_missing)

iris %>%
  skim() %>%
  focus(n_missing)

# This is equivalent to
iris %>%
  skim() %>%
  dplyr::select(skim_variable, skim_type, n_missing)"

downlit::evaluate_and_highlight(code)
#> Error in UseMethod("replay_html", x): pas de méthode pour 'replay_html' applicable pour un objet de classe "c('one_skim_df', 'tbl_df', 'tbl', 'data.frame')"

^{Created on 2022-04-19 by the reprex package (v2.0.1)}

[maelle]

Maybe r-lib/pkgdown#677 (comment) is a good thing to try (i.e. returning the object invisibly if not already the case)

[elinw]

Thank you for all of this. I'm going to try the invisibility approach. We do have custom objects and I guess something must have changed. We only build the site on a merge to main, so it has been a while.

[elinw]

@michaelquinn32 It looks to me like we need an explicit print.one_skim_df() function. I tried a bit but hit a dead end.

[elinw]

@michaelquinn32 I'm going to create a new issue, I thin we need to do a quick new release that I'll explain there.