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

Fix failing deployment caused by version error s3contents #4

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Lytrix
Copy link

@Lytrix Lytrix commented Feb 11, 2019

Got an installation error when I tried to deploy on Heroku. It was caused by a wrong version of a package in the pip.lock file.

s3contents was referring to version 0.2.2, which does not exist. Changed it to latest version 0.1.12 and updated all other packages as well. It now deploys as expected.

This PR does not fix the password not being recognised.

Lytrix and others added 2 commits February 11, 2019 23:27
Fixed non existing version number
@mordieross
Copy link

Not sure why this says i’ve Apparently Approved this as I never and am not able to do that anyway.

@mordieross
Copy link

But would like the version corrected so the button works again if any real approver can take look. Thanks

@Lytrix
Copy link
Author

Lytrix commented Feb 15, 2019

@kenneth-reitz could you take a look at my PR?

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