Skip to content
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

AphA CPD Survey data added #68

Merged
merged 13 commits into from
Aug 22, 2024
Merged

Conversation

francisbarton
Copy link
Contributor

  • Reproducible raw data import and cleaning script, with comments and context
  • Cleaning script in data-raw folder as suggested by {usethis}/"R Packages"
  • Clean column names and variable label attributes provided
  • Dataset documented (not to column level, but adequately?)

In response to issue #64.

@StatsRhian StatsRhian assigned StatsRhian and unassigned StatsRhian Aug 20, 2024
@StatsRhian
Copy link
Member

Happy to review this PR @francisbarton if you can get the checks passing. Looks like the logs are too old to be viewed. Worth a retrigger?

@francisbarton
Copy link
Contributor Author

Thanks @StatsRhian I'll have a look and see if I can find out why the checks are failing. It would be good to get this approved at long last. There was a thread about it on the NHS-R Slack but that is definitely lost to the mists of time now.

@StatsRhian
Copy link
Member

Happy to help debug the logs too when you rerun. But I because it's your fork I can't retrigger them to have a look.
Yeah I'm guessing that's lost slack-history.

@francisbarton
Copy link
Contributor Author

@StatsRhian Looks like the checks have passed now

@StatsRhian StatsRhian merged commit 8dbff91 into nhs-r-community:main Aug 22, 2024
2 checks passed
@francisbarton francisbarton deleted the issue64_fb branch August 26, 2024 19:24
@Lextuga007 Lextuga007 linked an issue Sep 28, 2024 that may be closed by this pull request
@Lextuga007
Copy link
Member

@all-contributors please add @francisbarton for code.

Copy link
Contributor

@Lextuga007

I've put up a pull request to add @francisbarton! 🎉

@Lextuga007
Copy link
Member

@all-contributors please add @StatsRhian for maintenance.

Copy link
Contributor

@Lextuga007

I've put up a pull request to add @StatsRhian! 🎉

@Lextuga007
Copy link
Member

@all-contributors please add @francisbarton for data

Copy link
Contributor

@Lextuga007

I've put up a pull request to add @francisbarton! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add survey data from AphA analysts
3 participants