-
Notifications
You must be signed in to change notification settings - Fork 26
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
Added a population dataset #95
Conversation
Co-authored-by: Tom Jemmett <[email protected]> Co-authored-by: Gabriel Hobro<[email protected]> Co-authored-by: Sarah Lucas<[email protected]>
Can you run devtools::document(), that should resolve the error. There's a note about the all contributors file @Lextuga007 this should be in .Rbuildignore |
I'm just fixing #86 😄 |
R/ons_uk_population_2023.R
Outdated
#' library(tidyverse) | ||
#' library(tidyr) | ||
|
||
#' Load the data in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#' Load the data in | |
#' # Load the data in |
that line needs to be forced to be a comment: it's trying to run Load the data in
as an R command
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
actually, I would remove and change the entire example here:
- the example is meant to be about using the data, not how to generate it
- the example includes a path to a file on your local computer
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, thats all been fixed just now anyway, but still failing !
@tomjemmett I've asked @jacgrout to push the changes she's made and I can also have a look to see what's failing. |
@all-contributors please add @jacgrout for data Thank you for contributing such a major PR as part of #Hacktoberfest! 🎉 |
I've put up a pull request to add @jacgrout! 🎉 |
Added a dataset of ONS Population Estimates for 2023 for the UK broken down by country, age, and sex
Closes #95