Mostly flake8, PEP8 and consistency changes. Also removed some unused or redundant checks in tests.
Also discovered an issue with the Shape class which assumes Path object rather than paths as string. This is not discovered in tests as the config variable is overwritten with a Path object. If the unmodified Config object is used, the code doesn't work. This can be resolved when the Data class is implemented and is not critical at this point.
Related Issue:
Suggested reviewer(s):
Reviewer checklist:
[x] The headers of all files contain a reference to the repository license
[x] 100% test coverage of new code
[x] All tests are passing
[x] All names (e.g., files, classes, functions, variables) are explicit
[x] Documentation (as docstrings) is complete and understandable
Summary:
Mostly flake8, PEP8 and consistency changes. Also removed some unused or redundant checks in tests.
Also discovered an issue with the Shape class which assumes Path object rather than paths as string. This is not discovered in tests as the config variable is overwritten with a Path object. If the unmodified Config object is used, the code doesn't work. This can be resolved when the Data class is implemented and is not critical at this point.
Related Issue:
Suggested reviewer(s):
Reviewer checklist: