-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Doc fix nitpick #15034
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
Doc fix nitpick #15034
Conversation
aka some missing references were fixed between the last build of the nitpicky-mode PR and its merge.
I pushed the fix for the offending flake line to your fork @tacaswell (hoping that it helps more than it troubles). In general, however, I have some problems with this nitpicky stuff.
So is anyone actually optimistic this can be solved anytime soon? If not, I would propose to disable nitpicky for now. |
@ImportanceOfBeingErnest I think you are right, but I would propose something slightly different: The line number bit was suggested by @anntzer in the original PR. I propose we remove the warnings about unused nitpick ignores by line number – but otherwise leave nitpick turned on. This will still prevent the addition of "new" incorrect references, but allow any existing missing references to persist. The disadvantage is that it will allow "old" missing reference ignores to persist as well. |
Thanks @ImportanceOfBeingErnest definitely helps more than it hurts! I'm going to merge this so that our docs build clean again and move discussion about this to a new issue. |
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 LGTM –
The double backtick issues is pervasive in matplotlib, and causes some issues in newer versions of numpydoc.
This replaces #15015 .
Closes #15030