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

Skip to content

Update for pydata-sphinx-theme 0.12.0 #24488

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
merged 1 commit into from
Nov 17, 2022
Merged

Conversation

ksunden
Copy link
Member

@ksunden ksunden commented Nov 17, 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.
@timhoffm timhoffm merged commit 45f8ebc into matplotlib:main Nov 17, 2022
@timhoffm
Copy link
Member

@meeseeksdev backport to v3.6.x

@lumberbot-app
Copy link

lumberbot-app bot commented Nov 17, 2022

Oops, something went wrong applying the patch ... Please have a look at my logs.

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