Fixes references outside function (#214)

[hoxbro]

Closes #214

[larsoner]

@sebp @MatthewFlamm can you see if this fixes #214 for you?

[MatthewFlamm]

It still fails on my usage, but I haven't compared precisely to the small test example that @hoxbro gave in #214. So there may be still some unresolved issue, or I'm using it incorrectly. It does work for me on 0.8.0 however.

[hoxbro]

@MatthewFlamm Can you give me a small code example of your problem? Then I can look into it.

[MatthewFlamm]

These days I unfortunately won't have time to try and simplify my case down to a simple example. Here is an example from how I used it though:

https://github.com/Merck/rtdpy/blob/master/source/AxialDispersion_cc.rst
https://github.com/Merck/rtdpy/blob/master/rtdpy/ad_cc.py

[hoxbro]

That's okay, the linked files were more than enough. I think that I have fixed your problem to:

@larsoner The more underlying problem, seem to be that the DEDUPLICATION_TAG is not necessary placed last in the sibling_section list. I have fixed this with a for-loop but I don´t know if there is a more underlying problem here.

[MatthewFlamm]

@hoxbro Great work! This works better than 0.8.0 even. I was getting some duplication of links in 0.8.0, but it was still understandable unlike 0.9+. Current behavior is below which works in my full code too.

[larsoner]

@hoxbro do you see some easy way to add a test? For example, you could add something to numpydoc/tests/tinybuild (it's a tiny sphinx-buildable thing) that fails to do the right thing on master, then assert that there is some suitable substring (or regex) that shows it's working on this PR.

[larsoner]

(the tests for tinybuild are in numpydoc/tests/test_full.py)

[larsoner]

Awesome, thanks @hoxbro !