aburrell / apexpy

A Python wrapper for Apex coordinates

MIT License

34 stars 25 forks source link

Deprecated Python 2 #70

Closed aburrell closed 3 years ago

aburrell commented 3 years ago

Description

Removes support for Python 2 and clarifies installation instructions.

Fixes #38 and records issue noted in #68.

Type of change

Breaking change (fix or feature that would cause existing functionality to not work as expected)
This change requires a documentation update

How Has This Been Tested?

Ran unit tests

Test Configuration:

Operating system: OS X Mojave
Python version number: 3.7
Compiler with version number: MacPorts gcc9 9.3.0_4

Checklist:

[x] Make sure you are merging into the develop (not main) branch
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] Add a note to Changelog.rst, summarising the changes
[x] Add yourself to AUTHORS.rst and .zenodo.json

coveralls commented 3 years ago

Pull Request Test Coverage Report for Build 729

34 of 35 (97.14%) changed or added relevant lines in 4 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 97.479%

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/apexpy/init.py	4	5	80.0%
<!--	Total:	34	35	97.14%	-->

Files with Coverage Reduction	New Missed Lines	%
src/apexpy/apex.py	1	97.68%
<!--	Total:	1		-->

Totals
Change from base Build 706:	-0.03%
Covered Lines:	369
Relevant Lines:	377

💛 - Coveralls

aburrell commented 3 years ago

Merging, since this is a straightforward PR and not a release.