aburrell / apexpy

A Python wrapper for Apex coordinates

MIT License

34 stars 25 forks source link

Codacy style #75

Closed aburrell closed 3 years ago

aburrell commented 3 years ago

Description

Improve code style and safety, fixes #60

Type of change

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Ran unit tests locally

Test Configuration:

Operating system: OS X Mojave
Python version number: 3.7
Compiler with version number: gcc version 9.3.0 (MacPorts gcc9 9.3.0_5)
Any details about your local setup that are relevant: branched from develop

Checklist:

[x] Make sure you are merging into the develop (not main) branch
[x] My code follows the style guidelines of this project
[ ] 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 779

4 of 4 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.95%

Totals
Change from base Build 763:	0.0%
Covered Lines:	373
Relevant Lines:	377

💛 - Coveralls

aburrell commented 3 years ago

Also evaluated several code patterns to ignore in codacy.