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

Update requests==2.8 in setup.py, too #31

Merged
merged 2 commits into from
Nov 11, 2015

Conversation

gnunicorn
Copy link

The changelog for the last release states, that request is now pinned at version 2.8, yet when installing the latest version of lassie, it requires (and install) version 2.6 – the setup.py hasn't been updated to reflect that change and breaks the installation. This PR corrects that.

@michaelhelmick
Copy link
Owner

Ah, thanks! Forgot to catch that. Mind updating test_requirements.txt as well.

Also, if you could do me one more solid, update them all to 2.8.1 rather than 2.8.0? :D

@gnunicorn
Copy link
Author

@michaelhelmick happy to do so.

@gnunicorn
Copy link
Author

@michaelhelmick made the setup.py read the requirements.txt instead. hope that is less error prone.

before you release the new version, I have problems running it here on python3.5 (will open a new ticket) – mind helping me figuring that out before?

michaelhelmick added a commit that referenced this pull request Nov 11, 2015
Update requests==2.8 in setup.py, too
@michaelhelmick michaelhelmick merged commit 1ba2f34 into michaelhelmick:master Nov 11, 2015
@michaelhelmick
Copy link
Owner

Sure, just post the stacktrace or any errors you're getting. Might not be able to help til later today. But I'll have a whack at it.

@gnunicorn gnunicorn deleted the patch-1 branch November 11, 2015 13:52
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