REFACTOR: Centralize all Config Interactions in Core

TravBots / hammer_tracker

0 stars 1 forks source link

REFACTOR: Centralize all Config Interactions in Core #28

Closed DMcConnell closed 5 days ago

DMcConnell commented 2 weeks ago

Step 1 in issue #25 - Move all config interactions into their own class

The major parts of the refactor are done, the code doesn't work though. Have some serious testing and cleaning up to do. We need unit tests!

DMcConnell commented 1 week ago

This now adds some unit tests as well and gets all tests to be passing. Also updates readme with some shortcut commands to run tests or start bot in dev mode.

DMcConnell commented 1 week ago

Discussed on Discord and revision requirements are:

Config access should be split out into it's own thing, stop passing the core around
Need to find the bug with difference between search and info links