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

Esgf test data fix #115

Merged
merged 11 commits into from
Jan 21, 2021
Merged

Esgf test data fix #115

merged 11 commits into from
Jan 21, 2021

Conversation

jhaigh0
Copy link
Collaborator

@jhaigh0 jhaigh0 commented Jan 21, 2021

Pull Request Checklist:

  • This PR addresses an already opened issue (for bug fixes / features)
    • This PR fixes issue #xyz
  • Tests for the changes have been added (for bug fixes / features)
  • Documentation has been added / updated (for bug fixes / features)
  • HISTORY.rst has been updated (with summary of main changes)
  • bumpversion minor has been called on this branch
  • I have added my relevant user information to AUTHORS.md

Switched from using individual fixtures for each test data set in the mini-esgf-data archive, to using one fixture or all esgf data tests which clones the repo if needed.
This was done because the quantity of individual file requests from using get_file() was causing github to reject some requests.

@ellesmith88 ellesmith88 self-requested a review January 21, 2021 13:13
@ellesmith88 ellesmith88 merged commit 99325eb into master Jan 21, 2021
@ellesmith88 ellesmith88 deleted the esgf-test-data-fix branch January 21, 2021 13:56
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

3 participants