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. |
alexrudy
left a comment
There was a problem hiding this comment.
This LGTM –
The double backtick issues is pervasive in matplotlib, and causes some issues in newer versions of numpydoc.
This replaces #15015 .
Closes #15030