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 installation with latest pip version #1104

Merged
merged 1 commit into from
May 6, 2019
Merged

Conversation

NaxAlpha
Copy link
Contributor

@NaxAlpha NaxAlpha commented Apr 9, 2019

Unable to install chalice with latest pip version

Hi, I have tried installing chalice (in a virtual environment). Whenever, I use pip install -r requirements.txt, it breaks with following error:

image

I have updated the code so that it will be compatible with latest version of pip and now it works smoothly. Let me know if any other change is required.

Thanks

@codecov-io
Copy link

Codecov Report

Merging #1104 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1104   +/-   ##
=======================================
  Coverage   95.47%   95.47%           
=======================================
  Files          27       27           
  Lines        4577     4577           
  Branches      579      579           
=======================================
  Hits         4370     4370           
  Misses        135      135           
  Partials       72       72

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 10d7fb5...385b0c6. Read the comment docs.

@stealthycoin stealthycoin mentioned this pull request May 6, 2019
@stealthycoin stealthycoin merged commit 385b0c6 into aws:master May 6, 2019
@stealthycoin
Copy link
Contributor

Thanks for the submission. I pushed a few extra commits on top of yours and merged this in #1115. We can't just remove the ceiling on PIP as explained #1085 by both @jamesls and @dtufft, in addition to that I added a changelog entry.

@NaxAlpha
Copy link
Contributor Author

NaxAlpha commented May 7, 2019

Thanks a lot, it works with the latest pip now.

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.

3 participants