When commits hit main, we run the gdformat tool, autoformat them, and commit those changes. This kinda sucks for a number of reasons.
However, it accomplishes the thing we care about most: enforcing consistent whitespace across the project in a way that doesn't put maintenance burden on moderators to explain how to fix it to devs (since Godot whitespace settings are per-editor, not per-project).
It would be great if we could solve that whitespace problem with a more elegant solution than creating a bot commit that enforces unchangeable formatting rules that we don't necessarily agree with.
For this in the current action, we should try gdformat ., see if we have the same issue as with gdformat "**/*.gd". It kinda defeats the purpose if we don't run the linter on half the code.
When commits hit main, we run the
gdformat
tool, autoformat them, and commit those changes. This kinda sucks for a number of reasons.However, it accomplishes the thing we care about most: enforcing consistent whitespace across the project in a way that doesn't put maintenance burden on moderators to explain how to fix it to devs (since Godot whitespace settings are per-editor, not per-project).
It would be great if we could solve that whitespace problem with a more elegant solution than creating a bot commit that enforces unchangeable formatting rules that we don't necessarily agree with.