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

Jump to bottom

Pin versions of psutil, gunicorn and werkzeug #190

Merged
merged 2 commits into from May 1, 2020
Merged

Pin versions of psutil, gunicorn and werkzeug #190

merged 2 commits into from May 1, 2020

Conversation

mermop
Copy link
Contributor

@mermop mermop commented May 1, 2020 •
edited

werkzeug and gunicorn have removed APIs relied upon by openfisca 30.
psutil 5.4.6 is required by openfisca 30.0.1

@mermop mermop requested a review from JacOng17 May 1, 2020 00:34
@mermop mermop changed the title Pin versions of psutil and gunicorn Pin versions of psutil, gunicorn and werkzeug May 1, 2020
JacOng17
JacOng17 approved these changes May 1, 2020
View reviewed changes
Copy link
Member

@JacOng17 JacOng17 left a comment •
edited

Choose a reason for hiding this comment

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

LGTM, needed these updates because they need to be pointing to the old APIs 👍

@mermop mermop merged commit d438c36 into master May 1, 2020
@mermop mermop deleted the psutil-version branch May 1, 2020 00:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JacOng17 JacOng17 JacOng17 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@mermop @JacOng17