-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Make it work on newer mkdocstrings, fix deprecation warnings #4626
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
base: master
Are you sure you want to change the base?
Conversation
| from pathlib import Path | ||
| from subprocess import PIPE, STDOUT, CalledProcessError, Popen | ||
| from typing import Any, List, Mapping, Optional | ||
| from typing import Any, ClassVar, TYPE_CHECKING, final, override |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CI complains about override:
ImportError: cannot import name 'override' from 'typing' (/usr/lib/python3.10/typing.py)
https://github.com/fmtlib/fmt/actions/runs/20028626137/job/57455771194
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
da3eef6 to
0cf82a5
Compare
| sudo apt update | ||
| sudo apt install doxygen | ||
| pip install mkdocs-material==9.5.25 mkdocstrings==0.26.1 mike==2.1.1 | ||
| pip install mkdocs-material==9.7.0 mkdocstrings==1.0.0 mike==2.1.3 typing_extensions==4.15.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are typing_extension required? In general it's good to keep the number of deps small.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is the backported runtime behavior for builtin typing. it's maintained by the python team and only has one file so i'm okay with it.
0cf82a5 to
fdad8cf
Compare
Tested on mkdocs 1.6.1 + mkdocstrings 1.0.0 + Python 3.14.0.
Ruff check target = py3.8