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

Pin to flask 1.1.2 #60

Merged
merged 2 commits into from
May 5, 2022
Merged

Pin to flask 1.1.2 #60

merged 2 commits into from
May 5, 2022

Conversation

nottrobin
Copy link
Contributor

@nottrobin nottrobin commented May 5, 2022

Flask is now more strict about jinja2, so this is no longer helpful.
pallets/flask@6d8b4ce

This will help us merge canonical/ubuntu-com-security-api#90.

QA

Look at canonical/ubuntu-com-security-api#90.

Flask v1.1.3 introduced upper bounds on a lot of dependencies:
pallets/flask@6d8b4ce

This has caused conflicts in our projects for Jinja2, click - e.g.:
https://github.com/canonical-web-and-design/ubuntu-com-security-api/runs/6309273393?check_suite_focus=true

For now, let's pin to the slightly older version of Flask.
@nottrobin nottrobin changed the title Unpin jinja2 Pin to flask 1.1.2 May 5, 2022
Copy link
Contributor

@jkfran jkfran left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@jkfran jkfran merged commit bf69a7a into main May 5, 2022
@jkfran jkfran deleted the unpin-jinja2 branch May 5, 2022 16:00
@nottrobin
Copy link
Contributor Author

I filed pallets/flask#4589

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