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

Refactor get_interventions_data #208

Merged
merged 3 commits into from
Mar 29, 2021
Merged

Refactor get_interventions_data #208

merged 3 commits into from
Mar 29, 2021

Conversation

kathsherratt
Copy link
Contributor

Interventions data stopped updating in December 2020. This function is for backwards-compatibility only. Function is now deprecated with warning.

This PR:

  • Adds lifecycle deprecated warning with link to Oxford policy tracker
  • Maintains usability for downloading; download/clean mostly uses existing code in main
  • Adds documentation explaining depreciation and link to alternatives

Fixes #164

- deprecate function with warning
- adds lifecycle deprecated warning with link to Oxford policy tracker
- retains use for downloading; download/clean mostly uses existing code
- adds documentation explaining depreciation and link to alternatives
@kathsherratt kathsherratt requested a review from seabbs March 27, 2021 00:39
R/get_interventions_data.R Show resolved Hide resolved
R/get_interventions_data.R Outdated Show resolved Hide resolved
Copy link
Contributor

@seabbs seabbs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Small style changes and function listed as internal? Needs to be added back to pkgdown and News item added highlighting depreciation of both functions and explaining why. Also need issue exploring what to do about this.

@kathsherratt
Copy link
Contributor Author

kathsherratt commented Mar 29, 2021

Ah thanks. On the docs I was following the lifecycle vignette:
image

but very happy to export as a normal function.

@seabbs
Copy link
Contributor

seabbs commented Mar 29, 2021

Ah I didn't know that worked like that. Sounds like a great idea!

Copy link
Contributor

@seabbs seabbs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me 😄

@kathsherratt kathsherratt merged commit f2cc466 into dev Mar 29, 2021
@kathsherratt kathsherratt deleted the refactor-interventions branch March 29, 2021 10:44
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.

2 participants