Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Backport PR#24488 to v3.6.2-doc #24507

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

Merged

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-doc milestone Nov 19, 2022
@oscargus oscargus merged commit f947512 into matplotlib:v3.6.2-doc Nov 19, 2022
@QuLogic QuLogic deleted the backport-of-pr24488-to-v3.6.2-doc branch November 22, 2022 04: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.

3 participants