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

Explicitly add jinja2 dependency #1031

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

craigds
Copy link
Contributor

@craigds craigds commented Oct 6, 2024

Overview

Fixes an error importing pycsw

Assuming you install the requirements from requirements-standalone.txt this was already implicitly pulled in, but this PR makes the dependency explicit

Related Issue / Discussion

Fixes #1030

Additional Information

Contributions and Licensing

(as per https://github.com/geopython/pycsw/blob/master/CONTRIBUTING.rst#contributions-and-licensing)

  • I have already previously agreed to the pycsw Contributions and Licensing Guidelines

This was implicitly pulled in by Flask previously, but is actually
required directly by pycsw so it should be explicitly mentioned in
requirements
@craigds craigds changed the title Add missing jinja2 dependency causing ModuleNotFoundError Explicitly add jinja2 dependency Oct 6, 2024
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.

Undeclared Jinja2 dependency
1 participant