Base Sphinx makefiles on the modern template

[jenshnielsen]

This significantly reduces the amount of boilerplate code in the makefiles and should hopefully make the logic a bit easier to follow

[codecov]

Codecov Report

Merging #1636 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1636      +/-   ##
==========================================
- Coverage   66.59%   66.58%   -0.01%     
==========================================
  Files         141      141              
  Lines       17406    17406              
==========================================
- Hits        11591    11590       -1     
- Misses       5815     5816       +1

[codecov]

Codecov Report

Merging #1636 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1636   +/-   ##
=======================================
  Coverage   66.58%   66.58%           
=======================================
  Files         141      141           
  Lines       17406    17406           
=======================================
  Hits        11590    11590           
  Misses       5816     5816

[astafan8]

Looks neat!

Could you add a link somewhere to the docs where this new way of interacting with sphinx is explained? please :)

[jenshnielsen]

@astafan8 I am not sure what you mean. Functionally there should be no big change. It is mainly that instead of having all these exilic targets on the line of target foo: call sphinx build with arg foo there is just a default target

[astafan8]

@jenshnielsen ah, got it now, thanks!