search

mchlnix / SMB3-Foundry

SMB3 Level Editor in Python
GNU General Public License v3.0
93 stars 12 forks source link

Actually run black during CI job #74

Closed fortenbt closed 3 years ago

fortenbt commented 3 years ago

Running the black command as given in the .travis.yml CI script produces the following:

$ black -l 120 --check
No Path provided. Nothing to do 😴

Running it with the foundry/ path gives:

black -l 120 --check foundry/
would reformat /mnt/c/development/fortenbt-foundry/foundry/game/gfx/objects/LevelObjectFactory.py
would reformat /mnt/c/development/fortenbt-foundry/foundry/game/gfx/objects/Jump.py
would reformat /mnt/c/development/fortenbt-foundry/foundry/game/gfx/drawable/Block.py
would reformat /mnt/c/development/fortenbt-foundry/foundry/game/gfx/objects/EnemyItem.py
would reformat /mnt/c/development/fortenbt-foundry/foundry/gui/AboutWindow.py
would reformat /mnt/c/development/fortenbt-foundry/foundry/gui/MainWindow.py
Oh no! 💥 💔 💥
6 files would be reformatted, 76 files would be left unchanged.

The branch that resolves this issue should also fix the code formatting in the given files.

mchlnix commented 3 years ago

Good catch. I was wondering, why that happened. I normally run it as a pre-commit hook.