Skip to content

Conversation

@QuLogic
Copy link
Member

@QuLogic QuLogic commented Nov 19, 2022

PR Summary

pydata-sphinx-theme updated, renaming some configuration names. (only one of which we actually configure)
While they are just warnings for now, we are configured to error on warning so CircleCI is indicating failure.

This does the 'easy' route of just changing to the new name and expecting the version in which it is implemented.

If desired, I could do a larger change to accomodate both by checking the version, but I'm also not too worried about more tightly pinning already relatively tight requirements.

PR Checklist

Documentation and Tests

  • [n/a] Has pytest style unit tests (and pytest passes)
  • Documentation is sphinx and numpydoc compliant (the docs should build without error).
  • [n/a] New plotting related features are documented with examples.

Release Notes

  • [n/a] New features are marked with a .. versionadded:: directive in the docstring and documented in doc/users/next_whats_new/
  • [n/a] API changes are marked with a .. versionchanged:: directive in the docstring and documented in doc/api/next_api_changes/
  • [n/a] Release notes conform with instructions in next_whats_new/README.rst or next_api_changes/README.rst

`pydata-sphinx-theme` updated, renaming some configuration names.
(only one of which we actually configure)
While they are just warnings for now, we are configured to error on
warning so CircleCI is indicating failure.

This does the 'easy' route of just changing to the new name and expecting
the version in which it is implemented.

If desired, I could do a larger change to accomodate both by checking
the version, but I'm also not _too_ worried about more tightly pinning
already relatively tight requirements.
@QuLogic QuLogic added this to the v3.6.3 milestone Nov 19, 2022
@tacaswell tacaswell merged commit 49d65a8 into matplotlib:v3.6.x Nov 19, 2022
@QuLogic QuLogic deleted the backport-of-pr24488 branch November 23, 2022 07:19
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.

3 participants