UPBGE / UPBGE-logicnodes

A Node-based Logic system for UPBGE 2.8+
Other
53 stars 18 forks source link

Refactor/make code dry #86

Open Teck-Freak opened 1 year ago

Teck-Freak commented 1 year ago

These refactorings should be api-compatible to the old version. It just makes code more dry

IzaZed commented 1 year ago

I like this idea! I didn't look at everything now but most of it seems to make sense

A lot of this code hasn't been touched in years, quite a bit of it is still left the way I initially found it

I'm currently working on breaking up the basicnodes.py and other files into smaller ones in an effort to clean up the whole addon - that makes it hard to just merge a lot of this into master, so the target branch should be "refactor_files"

I can look at incorporating your changes there if you don't want to do it yourself, but in any case I'd like to thank you for this!