Skip to content

Conversation

@bjlittle
Copy link
Member

@bjlittle bjlittle commented Sep 20, 2021

🚀 Pull Request

Description

This PR pulls the cartopy_feature_download.py script from the latest stable cartopy tag i.e., 0.20.0, during CI.

Closes #4306


Consult Iris pull request check list

@bjlittle bjlittle force-pushed the stable-cartopy-feature-download branch from 303f36b to aebaad4 Compare September 20, 2021 08:26
Copy link
Member

@rcomer rcomer left a comment

Choose a reason for hiding this comment

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

👍

@rcomer rcomer merged commit d1debf9 into SciTools:main Sep 20, 2021
tkknight added a commit to tkknight/iris that referenced this pull request Sep 22, 2021
* main: (94 commits)
  added support for make html-noapi and html-quick for the docs build (SciTools#4333)
  Refactor some netcdf save code (SciTools#4301)
  [pre-commit.ci] pre-commit autoupdate (SciTools#4329)
  Update to loading docs to cover absence of 'or' for constraints (SciTools#4321)
  update latest.rst.template (SciTools#4323)
  stable cartopy feature download (SciTools#4328)
  Skip TestConstrainedLoad if data missing (SciTools#4319)
  Add 'Good First Issue' label to reasons an issue doesn't go stale (SciTools#4317)
  Gallery: simplify quiver example (SciTools#4120)
  Improve styling in a minor way in docs (SciTools#4314)
  bump version (SciTools#4310)
  Made clear we only test on Linux. (SciTools#4309)
  Updated environment lockfiles (SciTools#4308)
  Include Discussions in Getting Involved. (SciTools#4307)
  Fixed text to show as link. (SciTools#4305)
  [pre-commit.ci] pre-commit autoupdate (SciTools#4299)
  Updated environment lockfiles (SciTools#4298)
  cartopy feature download (SciTools#4304)
  Mesh Loading (AVD-1813) (SciTools#4262)
  reset whatsnew latest (SciTools#4288)
  ...
@trexfeathers
Copy link
Contributor

How do we ensure this doesn't drift in future?

@bjlittle bjlittle deleted the stable-cartopy-feature-download branch January 21, 2022 10:15
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.

Update CI cartopy_cache cartopy URL

3 participants