-
Notifications
You must be signed in to change notification settings - Fork 224
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
Check and lint sphinx configuration file doc/conf.py #630
Conversation
The old way (the `setup()` function) is difficult to understand.
Co-authored-by: Wei Ji <[email protected]>
Do you know much about the lint error?
I can't reproduce it locally, but will update my dependencies and see what's up with that. |
Can't reproduce it with pylint 2.5.3, but CI uses pylint 2.6.0. |
This reverts commit f6ce407.
Nope, passes locally for me on |
This reverts commit e297833.
When I try to import an non-existent module like |
Description of proposed changes
Run flake8 and pylint to check and lint doc/conf.py.
It would be easier to review each commit separately.
Reminders
make format
andmake check
to make sure the code follows the style guide.doc/api/index.rst
.