Various refactors and tidy up.

wg4568 commented 3 years ago

The biggest change here is to the file structure.

game/ is for source code
game/lib is for internal libraries and APIs (this is a Python module, can be imported from game/)
res/ is for resources, static files etc. Will need to be subdivided into types as the project grows.
towergame.bat is just a shortcut for running game/main.py.

The second biggest change is using a code formatter. I went with Black. You can install it using pip install black and configure vscode to use it by running "format document" and opting to use Black (a popup in the bottom right). I would also recommend enabling "format on save" in settings.

wg4568 commented 3 years ago

Just a note, I made no changes to the code itself.

wg4568 commented 3 years ago

Just did some reading, we might want to configure our pylint.yml file to use Black also. It's a little more opinionated (and therefor more consistent) when compared to the default pylint stuff. It looks like @AdamJCavanaugh has done some work on this

AdamJCavanaugh commented 3 years ago

Yeah, I normally use flake8, but black would be fine. I'll plan to use that when I get around to working on the action.

jamesg31 / TowerGame

Various refactors and tidy up. #25