Moved config.past and config.future sets into class modules.

Brusalk / EventHorizon_Continued

EventHorizon_Continued for Legion+

MIT License

19 stars 15 forks source link

Moved config.past and config.future sets into class modules. #84

Closed Texl closed 6 years ago

Texl commented 6 years ago

I'd feel better having classes expose some subset of config data and letting ApplyConfig resolve the full application itself, but this would be a breaking change for existing user config overrides. (Something to avoid unless necessary IMO)

Fixes #50

Brusalk commented 6 years ago

Yay! Thanks for the PR for this! 🎉

Taking a look!

Brusalk commented 6 years ago

Thank you so much @Texl !!! Awesome PR 🎉