Skip to content

Conversation

@bjlittle
Copy link
Member

🚀 Pull Request

Description

This PR pins cftime<1.3.0. The iris tests are broken with the latest released version 1.3.0, which pulls from PyPI.

Pinning will give us time to investigate and either fix iris and/or feedback to the cftime developers.


Consult Iris pull request check list

@pp-mo
Copy link
Member

pp-mo commented Nov 23, 2020

See errors investigation : #3923

@trexfeathers
Copy link
Contributor

Given the long wait times for Travis at the moment, the simplicity of this change, and @rcomer 's observation in #3923 (comment), I'm going to merge this without waiting for Travis 😬

@trexfeathers trexfeathers merged commit a4d42f1 into SciTools:master Nov 23, 2020
@bjlittle
Copy link
Member Author

I've got a pull request migrating Travis-CI to Cirrus-CI waiting in the wings... might have to prioritise some effort on that now as Travis-CI is simply unusable as it stands.

@jonseddon
Copy link
Contributor

What's the easiest way for this commit to also be merged into the v3.0.x branch, as the tests are currently failing there for the same reason?

@trexfeathers
Copy link
Contributor

trexfeathers commented Nov 24, 2020

What's the easiest way for this commit to also be merged into the v3.0.x branch, as the tests are currently failing there for the same reason?

See this change in #3928

#3928 (comment)

@bjlittle bjlittle deleted the pin-cftime branch November 25, 2020 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants