FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Expire the _rename_parameters API changes. by anntzer · Pull Request #16491 · matplotlib/matplotlib · GitHub

Repository navigation

Expire the _rename_parameters API changes. - #16491

Merged
story645 merged 1 commit into
matplotlib:masterfrom
anntzer:rename_parameters
Feb 25, 2020
Merged

story645 merged 1 commit into
matplotlib:masterfrom
anntzer:rename_parameters

Conversation

anntzer commented Feb 13, 2020

Copy link
Copy Markdown
Contributor

I put the changelog not under "removals", though they could possibly go
under "behavior changes" instead?

PR Summary

PR Checklist

  • Has Pytest style unit tests
  • Code is Flake 8 compliant
  • New features are documented, with examples if plot related
  • Documentation is sphinx and numpydoc compliant
  • Added an entry to doc/users/next_whats_new/ if major new feature (follow instructions in README.rst there)
  • Documented in doc/api/api_changes.rst if API changed in a backward-incompatible way

anntzer added this to the v3.3.0 milestone Feb 13, 2020

jklymak left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

CI failure seems real though...

timhoffm commented Feb 16, 2020 •
edited
Loading

Copy link
Copy Markdown
Member

Sphinx has three warnings:

/usr/local/lib/python3.8/importlib/__init__.py:127: MatplotlibDeprecationWarning: 
The mpl_toolkits.axes_grid1.colorbar module was deprecated in Matplotlib 3.2 and will be removed two minor releases later. Use matplotlib.colorbar instead.
reading sources... [ 12%] api/_as_gen/matplotlib.axes.Axes.secondary_xaxis
<string>:6: RuntimeWarning: divide by zero encountered in true_divide
/home/circleci/project/doc/api/next_api_changes/removals.rst:150: WARNING: py:obj reference target not found: bezier

of which i think only the last one really counts for the failure (build finished with problems, 1 warning.)

anntzer commented Feb 16, 2020 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Added an api entry for matplotlib.bezier to resolve the missing ref.
edit: ... which meant fixing some markup in that module as well.

I put the changelog not under "removals", though they could possibly go
under "behavior changes" instead?
story645 merged commit dc1f0d9 into matplotlib:master Feb 25, 2020
anntzer deleted the rename_parameters branch February 25, 2020 21:30
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL