Test and work properly on sphinx < 1.7

[njsmith]

Also adds testing on 3.7

Fixes gh-15

[codecov]

Codecov Report

Merging #16 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #16   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           3      3           
  Lines         264    267    +3     
  Branches       42     42           
=====================================
+ Hits          264    267    +3

Impacted Files	Coverage Δ
sphinxcontrib_trio/__init__.py	100% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3b7d19d...8e39125. Read the comment docs.

[Sraw]

I guess it works fine now :)

[njsmith]

@Sraw oh awesome!

....okay now I'm confused though. Do you know why my first version passed the tests? I forgot to disable the test you added to make sure that the ordering worked, so it should have failed, I think...

[Sraw]

I am confused too. Let me have a look.

[Sraw]

Funny, it works out of the box lol. Seems my modification is redundant.
I've searched a lot and cannot fully understand why. Guess sphinx==1.6.7 cannot really distinguish normal function and async funtion.

I will dive in further when I am free.

[njsmith]

Huh. Yeah, it is possible that the original bug only affected sphinx 1.7 and we never noticed until now, I could believe that.

[Sraw]

Yep, as our test never covers sphinx~=1.6.0 at all before.
Now it is fine. I guess the modification can be removed.

[njsmith]

OK that was a lot more confusing than I expected but I think we've got it nailed down now! I was impressed by how quickly you figured out the AttrDocVisitor trick, even if it turned out not to be needed after all :-)

[Sraw]

Congratulation :-)