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 uplink with Python 2.7 #223

Merged
merged 22 commits into from
Dec 31, 2021
Merged

Fix uplink with Python 2.7 #223

merged 22 commits into from
Dec 31, 2021

Conversation

shadchin
Copy link
Contributor

@shadchin shadchin commented Apr 2, 2021

Fixes #217 .

Changes proposed in this pull request:

  • Return support Python 2.7

Attention: @prkumar

@codecov
Copy link

codecov bot commented Apr 2, 2021

Codecov Report

Merging #223 (fc21923) into master (d3ee283) will increase coverage by 0.08%.
The diff coverage is 100.00%.

❗ Current head fc21923 differs from pull request most recent head 0babd17. Consider uploading reports for the commit 0babd17 to get more accurate results
Impacted file tree graph

@@             Coverage Diff             @@
##           master      #223      +/-   ##
===========================================
+ Coverage   99.91%   100.00%   +0.08%     
===========================================
  Files          44        44              
  Lines        2398      2410      +12     
  Branches      182       342     +160     
===========================================
+ Hits         2396      2410      +14     
+ Misses          2         0       -2     
Impacted Files Coverage Δ
uplink/arguments.py 100.00% <100.00%> (ø)
uplink/auth.py 100.00% <100.00%> (ø)
uplink/commands.py 100.00% <100.00%> (ø)
uplink/compat.py 100.00% <100.00%> (ø)
uplink/converters/__init__.py 100.00% <100.00%> (ø)
uplink/converters/keys.py 100.00% <100.00%> (ø)
uplink/converters/typing_.py 100.00% <100.00%> (ø)
uplink/decorators.py 100.00% <100.00%> (ø)
uplink/hooks.py 100.00% <0.00%> (ø)
uplink/types.py 100.00% <0.00%> (ø)
... and 4 more

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 d3ee283...0babd17. Read the comment docs.

@prkumar
Copy link
Owner

prkumar commented Dec 31, 2021

I added Python 2.7 to travis CI config, but the build is failing seemingly because of the issue mentioned here: pypa/pipenv#4357 (comment)

@prkumar prkumar merged commit a660529 into prkumar:master Dec 31, 2021
@shadchin shadchin deleted the fix_27 branch January 1, 2022 06:01
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.

uplink 0.9.4 is no longer compatible with Python 2.7.5
2 participants