Need to fix up sphinx warnings, etc.#231
Conversation
|
Oh yeah, this was pointed out in #229 |
|
Down to 2 warnings: @eteq, @astrofrog - is there something genuinely missing, or should I find a way to drop this intersphinx link? |
|
https://travis-ci.org/keflavich/astroquery/jobs/16460532
No idea.
|
|
@keflavich re: the configuration item stuff: I noticed similar warnings while doing #269 - there it was caused by an import of re: re: Running locally, I also noticed some |
|
After posting that, I realized it was probably easier to just fix them, since I went to the trouble of finding the issues. So keflavich#5 has a commit that leads to all the warnings going away, at least locally for me. So @keflavich, maybe merge that and see if the tests pass here? |
|
Thanks @eteq, I'll try the merge and test locally. You've caught a lot of small things that are caused by astroquery's mixed heritage (i.e., many independent committers). I didn't know about the |
|
Oh, I totally understand the cause - that's why I turned on the sphinx test in astropy in the first place - its about the only way to keep people honest about it :). Apparently the test runs in your repo too - keflavich#5 is passing (and includes the sphinx build), so that probably means if you merge it this will be good to go too. |
Documentation fixes to eliminate sphinx warnings
|
Hooray, it worked! |
Need to fix up sphinx warnings, etc.
Sphinx gives a boatload of warnings each time we build (e.g. https://travis-ci.org/astropy/astroquery/jobs/11808153). These don't cause any major problems I'm aware of, but they should still be cleaned up