-
Notifications
You must be signed in to change notification settings - Fork 111
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Wording in import section of packages #77
Comments
Just getting back to this now, @mine-cetinkaya-rundel! The distinction that I can see is that they're not tidyverse repos: httr and xml2 are in r-lib, jsonlite is Jeroen's, rvest is still a hadley repo, and DBI isn't on the tidyverse repo list to which I believe you're referring. None of the above is to say that it's clear to anyone reading the site; but, given they also don't have tidyverse pages (package.tidyverse.org), I can imagine it being slightly confusing not to differentiate at all. That said, I'm not opposed to changing it as you've described at all. Just kind of digging around to figure things out. |
Yeah, maybe we should move all those packages into an affiliates section? Or friends maybe? They're installed with the tidyverse package, but they're not members of the tidyverse organisation. Or is that another layer that's just too confusing? |
Oh, interesting. I didn't realise that blob, hms, modelr, and broom didn't have associated pages, so at least one part of my reasoning has crumbled. |
I think with a brief definition of the layers on top of the page, another layer wouldn't necessarily make things more confusing. |
Currently it says "There are a handful of other packages that are not in the tidyverse, but are tidyverse-adjacent. They are very useful for importing data from other sources:" and lists jsonlite, xml2, httr, rvest, and DBI under this. It's unclear to me what the difference of these from readxl and haven is since all of these packages are installed with tidyverse but are not in the core. It might be more clear to list all in the same hierarchy (as is the case in the tidyverse repo README).
The text was updated successfully, but these errors were encountered: