-
-
Notifications
You must be signed in to change notification settings - Fork 18.6k
DOC: Fix errors detected by sphinx-lint #46586
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
Conversation
Nice, thanks - could you add this to https://github.com/pandas-dev/pandas/blob/672798ec7b5565366be24aa4f57f6374f3a5e0fd/.pre-commit-config.yaml |
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.
Really nice clean up. I thought we were already validating this in the CI, but since it's not will be great to have it added to precommit
as said. But I think this can be in a follow up PR.
Thanks a lot!
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.
I don't understand the error :( |
looks unrelated, don't worry about it |
Should I merge main to get it fixed (in other words, do I have something to do?) |
I re-ran and it passed - all good, thanks! |
* Avoid mixing tabs and space. * fix rst issues found by sphinx-lint. * Make flake8-rst happy. * FIX: two hyperlinks in rst. * Marc' suggestion * Use sphinx-lint tag in pre-commit hook.
Found a few issues on the doc with sphinx-lint.
Fixed them.