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

Drop support for Python 2.7 #45

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

Conversation

wrightky
Copy link
Collaborator

Conda deployment failed, seems to be due to an attempted install on Python 2.7. Dropping all remaining mentions of the legacy version I could find.

@codecov
Copy link

codecov bot commented Jun 20, 2023

Codecov Report

Merging #45 (842a8d6) into master (8c90ac9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           master      #45    +/-   ##
========================================
  Coverage   97.66%   97.66%            
========================================
  Files           4        2     -2     
  Lines        1114      557   -557     
========================================
- Hits         1088      544   -544     
+ Misses         26       13    -13     

see 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@wrightky wrightky requested a review from elbeejay June 20, 2023 22:49
Copy link
Member

@elbeejay elbeejay left a comment

Choose a reason for hiding this comment

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

Looks fine to me

@elbeejay
Copy link
Member

Since pypi deployment did succeed yesterday, the version was updated to 2.5.3. https://pypi.org/project/pydorado/2.5.3 (although it still states support for 2.7)

@elbeejay
Copy link
Member

elbeejay commented Jul 5, 2023

@wrightky feel free to merge anytime, my comment above was that you could potentially bump the version one minor increment to make sure the pypi page/deployment incorporates these changes if you plan to tag a release

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