Closed thepetk closed 10 months ago
Restructures project in order to be more scalable. The new structure contains multiple different modules:
configuration
exceptions
mockers
parsers
JsonParser
tests
utils
validators
Other than that, it also adds a Makefile with run, format and test commands. Additionally multiple comments and docstrings are added to the code.
Makefile
run
format
test
fixes https://github.com/thepetk/projectman/issues/22
Testing and documentation do not need to be complete in order for this PR to be approved. We just need to ensure tracking issues are opened.
[ ] Unit/Functional tests
[ ] Documentation
What does this PR do?
Restructures project in order to be more scalable. The new structure contains multiple different modules:
configuration: all configuration related classes belong there.exceptions: all errors.mockers: mocked values and classes.parsers: configuration file parsers (e.gJsonParser)tests: all tests.utils: all utils that are used in different places of code.validators: FieldValidators etc.Other than that, it also adds a
Makefilewithrun,formatandtestcommands. Additionally multiple comments and docstrings are added to the code.Which issue(s) does this PR fix
fixes https://github.com/thepetk/projectman/issues/22
PR acceptance criteria
Testing and documentation do not need to be complete in order for this PR to be approved. We just need to ensure tracking issues are opened.
[ ] Unit/Functional tests
[ ] Documentation
How to test changes / Special notes to the reviewer