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

Remove deprecations #1897

Merged
merged 7 commits into from
Oct 8, 2021
Merged

Remove deprecations #1897

merged 7 commits into from
Oct 8, 2021

Conversation

greglucas
Copy link
Contributor

This PR removes a lot of deprecated code, which had alternatives and warnings associated with it.

Deprecations I've left for now:

  • gridliner attributes as I don't think we are fully settled on what the right path forward is there.
  • GOOGLE_MERCATOR - This one only got a release note change (albeit a while ago), but I'm not sure we really need to deprecate it even...

lib/cartopy/crs.py Outdated Show resolved Hide resolved
@QuLogic QuLogic added this to the 0.21 milestone Oct 7, 2021
The deprecation warnings have been around for several years now
with alternatives given, so remove these from the code base.
Users can call add_feature instead. This has been deprecated for
several years now.

There are other tests that test the add_feature interface already,
so just remove this test as it isn't really testing anything new.
It has been deprecated for several years now.
It has been deprecated for several years now.
Users should call Stamen('terrain-background') now.

Moved the references up to the new section.
These have been deprecated for several years now.
It has been deperecated for several years and has an alternative
in matplotlib.
These should use Mercator.GOOGLE now instead.
@QuLogic QuLogic merged commit 78ea7f8 into SciTools:master Oct 8, 2021
@greglucas greglucas deleted the remove-deprecations branch November 14, 2021 14:29
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