teragrep / cfe_31

0 stars 0 forks source link

Dissect Configuration class #12

Closed MoonBow-1 closed 7 months ago

MoonBow-1 commented 8 months ago

The Configuration class should be divided into smaller classes, so they can be passed to other methods/classes more easily and efficiently.

Would be looking at a maximum of 8 classes.

MoonBow-1 commented 8 months ago

Configuration has been now refactored in commit 18287d78 on refactor-configuration branch

MoonBow-1 commented 8 months ago

Individual classes are now passed as arguments instead of the whole Configuration in commit 17922c6b

MoonBow-1 commented 8 months ago

Also missed some methods in the new classes, added them in commit 510c8b7e

MoonBow-1 commented 8 months ago

Going to add a class for PropertiesFile, as well as remove duplicate code from tests for Configurations

MoonBow-1 commented 8 months ago