Skip to content

Conversation

@alexamici
Copy link
Collaborator

Depending on the kind of problem python-eccdodes may return RuntimeError or ImportError.

@keewis
Copy link
Collaborator

keewis commented Apr 11, 2021

looks good to me. Do you want to also pin eccodes temporarily? Otherwise the cfgrib tests will be skipped.

@keewis
Copy link
Collaborator

keewis commented Apr 11, 2021

@andersy005, do you know of a way to skip running on push if pull_request is also run? Otherwise every PR opened from a branch on the main repository will have two CI runs.

Edit: would limiting push to master and maybe stable work?

@alexamici
Copy link
Collaborator Author

I'll merge it now to stop the CI from failing. The real fix is on the python-eccodes side: conda-forge/python-eccodes-feedstock#69 and conda-forge/python-eccodes-feedstock#71

@alexamici alexamici merged commit 851d85b into master Apr 12, 2021
@alexamici alexamici mentioned this pull request Apr 12, 2021
@keewis keewis deleted the alexamici-patch-1 branch April 12, 2021 11:24
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