BUG: Connect to earlier event

[larsoner]

Closes #268.

Another option might be to still connect to build-inited but with a lower priority number, but in theory connecting to config-inited should be okay (and arguably cleaner anyway).

[jnothman]

We currently have CI testing on Sphinx 1.6.5 which does not know config-inited. Do we end support for it (I suppose that's not the right thing to do in a patch release), or switch hooks for Sphinx <2

[larsoner]

I can just try/except for now and make a note that we should get rid of it once we bump our Sphinx requirement

[codecov-commenter]

Codecov Report

Merging #269 into master will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #269      +/-   ##
==========================================
+ Coverage   92.66%   92.70%   +0.03%     
==========================================
  Files           7        7              
  Lines        1255     1261       +6     
==========================================
+ Hits         1163     1169       +6     
  Misses         92       92              

[jnothman]

This is good by me, but deserves confirmation from @arokem

[arokem]

This works on my end. The master branch still raises the error mentioned in the issue, but with this branch installed, no error. Thanks!

[jnothman]

Thanks @larsoner and @arokem