Added scons tool to run black automatically during the build

[dmoody256]

Runs black automatically on the git changeset, and only if the files have changed since the last time it ran.

[Calinou]

We already have a pre-commit hook for black: https://github.com/godotengine/godot/blob/master/misc/hooks/pre-commit-black

Also, in the future, I would prefer switching to a more established tool with #46235. This would make it easier for new contributors to adopt pre-commit hooks 🙂

[dmoody256]

We already have a pre-commit hook for black: https://github.com/godotengine/godot/blob/master/misc/hooks/pre-commit-black

How do I make that work automatically?

[Calinou]

How do I make that work automatically?

Follow the instructions in the documentation. In short, you need to copy all the files from the repository's misc/hooks to your local .git/hooks/ folder located at the repository root. When you make a commit, the pre-commit hooks will run automatically. Use git commit --no-verify to skip pre-commit hooks if needed.

Switching to pre-commit would make this step easier, since you'd only have to run pip3 install --user pre-commit && pre-commit install at the repository root to get started. Partial commits are also handled automatically by pre-commit.

[dmoody256]

How do I make that work automatically?

Follow the instructions in the documentation. In short, you need to copy all the files from the repository's misc/hooks to your local .git/hooks/ folder located at the repository root. When you make a commit, the pre-commit hooks will run automatically. Use git commit --no-verify to skip pre-commit hooks if needed.

Switching to pre-commit would make this step easier, since you'd only have to run pip3 install --user pre-commit && pre-commit install at the repository root to get started. Partial commits are also handled automatically by pre-commit.

The pre-commit hook did not run on my new files in the ninja tool pr #52793, is that intended?

[YuriSizov]

Not sure why a pre-commit hook didn't run for you, but it should, so there is no reason for this enhancement. Drop by the contributors chat if you need help figuring your issue out. Thanks for your contribution nonetheless.