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

Two small changes #39

Merged
merged 3 commits into from
Aug 6, 2021
Merged

Two small changes #39

merged 3 commits into from
Aug 6, 2021

Conversation

emmamendelsohn
Copy link
Contributor

In case this is helpful to you:

  • ots_create_tidy_data(), change the default value for reporters to "all" instead of "usa", per the function description.
  • ots_create_tidy_data_unmemoised() allow max year beyond 2018

@pachadotdev
Copy link
Collaborator

thanks a lot @emmamendelsohn !!
I'm updating the database with a new methodology, please give me 1 week to change that first

@pachadotdev pachadotdev merged commit dba8502 into ropensci:master Aug 6, 2021
@pachadotdev
Copy link
Collaborator

@emmamendelsohn I still have problems with the laptop but I was able to merge this anyways :)
thanks a lot

@emmamendelsohn
Copy link
Contributor Author

Thanks @pachadotdev !

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.

None yet

2 participants