Skip to content

pkgdown site for broom#260

Merged
alexpghayes merged 2 commits intotidymodels:masterfrom
jayhesselberth:pkgdown
Jun 4, 2018
Merged

pkgdown site for broom#260
alexpghayes merged 2 commits intotidymodels:masterfrom
jayhesselberth:pkgdown

Conversation

@jayhesselberth
Copy link
Contributor

@jayhesselberth jayhesselberth commented Nov 19, 2017

broom needs a pkgdown site!

Preview at https://vigilant-goldberg-0fd2e5.netlify.com/

  • add the broom logo as logo.png in man/figures and re-run pkgdown::build_site()
  • create docs/CNAME with single line containingbroom.tidyverse.org
  • In Settings / Github Pages / Source choose "master branch /docs folder"

The current Reference page has a lot of redundant entries because of the extensive use of S3 in broom (see r-lib/pkgdown#436).

@jayhesselberth
Copy link
Contributor Author

jayhesselberth commented Nov 19, 2017

closes #238, closes #272

@topepo
Copy link
Member

topepo commented Nov 20, 2017

You might consider using DT for the long table (or put it in an article)

@alexpghayes alexpghayes merged commit 3c8c114 into tidymodels:master Jun 4, 2018
@alexpghayes
Copy link
Collaborator

Thanks!

@alexpghayes
Copy link
Collaborator

@batpigandme I don't have access to the repo settings. Can you set up the Github pages?

@batpigandme
Copy link
Contributor

@alexpghayes 10-4, I'm separated from my laptop rn, but I'll be on it once we're ruinited (my laptop and I) later tonight or AM. 👍

@batpigandme
Copy link
Contributor

batpigandme commented Jun 5, 2018

Oh, shoot do you mean on netlify, @alexpghayes? If so, I don't have access to that either…

Disregard, should be good.

@alexpghayes
Copy link
Collaborator

Thanks!

@github-actions
Copy link

This pull request has been automatically locked. If you believe you have found a related problem, please file a new issue (with a reprex: https://reprex.tidyverse.org) and link to this issue.

@github-actions github-actions bot locked and limited conversation to collaborators Mar 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants