Refactoring
Description
After setting up basic CI with linting, need in refactoring became evident.
The code is brought in accordance with PEP8 standard and few improvements were also made,
notably lazy logging is now used, so the logger should not bring too much overhead
(this should be visible with test bages). The CI file was pre-pushed to the develop in order to have convenient badges on the gitlab page. Sorry
Related Issue
Closes #12 (closed)
How Has This Been Tested?
Tested both locally and with the CI. The pylint score changed from 4.06 to 9.82
Types of changes
-
Bug fix (non-breaking change which fixes an issue) -
New feature (non-breaking change which adds functionality) -
Breaking change (fix or feature that would cause existing functionality to change)
Checklist:
-
My code follows the code style of this project. -
My change requires a change to the documentation. -
I have updated the documentation accordingly. -
I have read the CONTRIBUTING document. -
I have added tests to cover my changes. -
All new and existing tests passed.