guzzle / guzzle_sphinx_theme

Sphinx theme used by Guzzle
MIT License
169 stars 60 forks source link

Use Sphinx.set_translator #23

Closed tbekolay closed 7 years ago

tbekolay commented 7 years ago

Hi! Thanks for this Sphinx theme, it looks beautiful.

I tried it with the most recent version of Sphinx and got a deprecation warning on account of the use of the html_translator_class config option. The recommended way to do this now is to call the set_translator method on the Sphinx app, so this PR does that. I believe this method is available in any version of Sphinx >= 1.3, but since the current requirements had < 1.3 I assumed there was an issue with that release and so set the required version to be > 1.3.

mtdowling commented 7 years ago

Looks good, thanks!