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

Added specific version range for jinja2 #26

Conversation

sanderdriesen
Copy link
Contributor

Due to changes in the latest released jinja2 the project is failing to run.
The following exception is thrown:

Attaching to docker_bot_1
bot_1 | Traceback (most recent call last):
bot_1 | File "run_test.py", line 4, in
bot_1 | from youengine.helpers.analyze import analyze_mpl, analyze_bokeh
bot_1 | File "/opt/bot/youengine/helpers/analyze.py", line 4, in
bot_1 | import bokeh.plotting
bot_1 | File "/usr/local/lib/python3.8/site-packages/bokeh/plotting/init.py", line 67, in
bot_1 | from ..document import Document; Document
bot_1 | File "/usr/local/lib/python3.8/site-packages/bokeh/document/init.py", line 35, in
bot_1 | from .document import DEFAULT_TITLE ; DEFAULT_TITLE
bot_1 | File "/usr/local/lib/python3.8/site-packages/bokeh/document/document.py", line 49, in
bot_1 | from ..core.templates import FILE
bot_1 | File "/usr/local/lib/python3.8/site-packages/bokeh/core/templates.py", line 42, in
bot_1 | from jinja2 import Environment, FileSystemLoader, Markup
bot_1 | ImportError: cannot import name 'Markup' from 'jinja2' (/usr/local/lib/python3.8/site-packages/jinja2/init.py)
docker_bot_1 exited with code 1
By adding a specific version range in requirements.txt for jinja2, the issue is fixed.

Jinja2>=2.10.1,<3.1

@ridlab ridlab marked this pull request as ready for review May 23, 2022 15:39
@ridlab ridlab merged commit f3a45ca into RevenyouIO:master May 23, 2022
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 this pull request may close these issues.

2 participants