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

Upgrade packages with pur #4124

Merged
merged 1 commit into from
Jun 7, 2018
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 7 additions & 10 deletions requirements/pip.txt
Original file line number Diff line number Diff line change
@@ -1,13 +1,10 @@
# Base packages
pip==10.0.0
pip==10.0.1
appdirs==1.4.3
virtualenv==15.2.0
virtualenv==16.0.0
docutils==0.14
Sphinx==1.7.4

# 0.3.0 breaks sidebar
# https://github.com/rtfd/sphinx_rtd_theme/issues/610
sphinx_rtd_theme==0.2.5b1
sphinx_rtd_theme==0.3.1

Pygments==2.2.0

Expand All @@ -24,7 +21,7 @@ readthedocs-build<2.1
django-tastypie==0.13.0

django-guardian==1.4.9
django-extensions==2.0.6
django-extensions==2.0.7

# djangorestframework 3.7.x drops support for django 1.9.x
djangorestframework==3.6.4
Expand All @@ -39,7 +36,7 @@ defusedxml==0.5.0

# Basic tools
redis==2.10.6
celery==4.1.0
celery==4.1.1

# django-allauth 0.33.0 dropped support for Django 1.9
# https://django-allauth.readthedocs.io/en/latest/release-notes.html#backwards-incompatible-changes
Expand All @@ -49,7 +46,7 @@ dnspython==1.15.0

# VCS
httplib2==0.11.3
GitPython==2.1.9
GitPython==2.1.10

# Search
elasticsearch==1.5.0
Expand All @@ -74,7 +71,7 @@ django-formtools==2.1
# docker is pinned to 3.1.3 because we found some strange behavior
# related to timeouts on EXEC with 3.2.1 that's not present with 3.1.3
# https://github.com/rtfd/readthedocs.org/issues/3999
docker==3.1.3
docker==3.3.0
Copy link
Member

Choose a reason for hiding this comment

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

Are we sure the above comment is fixed? I'm not sure upgrading all our dependencies all the time is the best way to ensure stability, sadly :/

Copy link
Member Author

Choose a reason for hiding this comment

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

We don't.

I tested this locally and everything was working. Although, last time it happened the same, but the problem appeared only in production and we weren't able to reproduce locally.

I'm not sure how I can do a test this in production without breaking everything. Maybe just install this in one of the builders, test it and then undo it?

Copy link
Member

Choose a reason for hiding this comment

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

I think hopefully it should be fixed. There isn't a great way to test in prod, other than to deploy it sadly. :/


django-textclassifier==1.0
django-annoying==0.10.4
Expand Down