Skip to content

Conversation

@lbdreyer
Copy link
Member

@lbdreyer lbdreyer commented Feb 19, 2020

Previously a couple tests were failing with eccodes> 2.14 as eccodes was no longer raising an error when you give it a mock object. This fixes those tests.

The environment currently resolves with eccodes 2.16

@lbdreyer lbdreyer requested a review from pp-mo February 19, 2020 09:58
@coveralls
Copy link

Coverage Status

Coverage remained the same at 86.777% when pulling 19cf20c on lbdreyer:latest_grib into 1f859fc on SciTools:master.

dependencies:
- iris>=2
- python-eccodes>=0.9.1,<2
- python-eccodes
Copy link
Member

Choose a reason for hiding this comment

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

Does this mean they have fixed the version-going-backwards problem ??? 😛

@pp-mo pp-mo merged commit a09fb02 into SciTools:master Feb 19, 2020
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.

3 participants