Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecation warning about javascript #45

Open
erikkemperman opened this issue Jun 4, 2019 · 2 comments
Open

Deprecation warning about javascript #45

erikkemperman opened this issue Jun 4, 2019 · 2 comments

Comments

@erikkemperman
Copy link

Hi,

I am getting deprecation warnings about javascript:

venv/lib/python3.7/site-packages/guzzle_sphinx_theme/guzzle_sphinx_theme/layout.html:198: RemovedInSphinx30Warning: To modify script_files in the theme is deprecated. Please insert a <script> tag directly in your theme instead.

Sphinx version 2.0.1
Guzzle theme version 0.7.11

@ronankeane
Copy link

Hi.

I'm getting this warning too, just installed theme today. Do I need to be concerned? Thanks.

c:\python37\lib\site-packages\guzzle_sphinx_theme\guzzle_sphinx_theme\layout.html:198: RemovedInSphinx30Warning: To modify script_files in the theme is deprecated. Please insert a <script> tag directly in your theme instead.

Sphinx Version 2.2.0

@bruceravel
Copy link

I am by no means a sphinx expert, so there might be repercussions to my suggestion.

I removed line 5 from the file guzzle_sphinx_theme/layout.html. This one:

  {% set script_files = script_files + ["_static/myscript.js"] %}

This appears to be using a soon-to-be-deprecated method of including js in your project. If you are not using a file explicitly called myscript.js in the _static/ folder, then removing that line seems to have no impact. And it gets rid of that warning.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants