Pre-commit

robert lippmann <[email protected]> Fri, 16 May 2025 10:32:44 -0400
Newsgroups gmane.comp.emulators.wine.devel
Message-ID <[email protected]>
Hi all,

I was looking in the docs, and came across this (https://gitlab.winehq.org/wine/wine/-/wikis/Git-Wine-Tutorial#removing-trailing-whitespace):

"It fixes whitespace on all the commits that you've made. I think it's
pretty good about merge conflicts due to whitespace as well. I don't
know of a way of doing this at commit time though.”

Well, actually there is.  Pre-commit (https://pre-commit.com/).  Actually fixing whitespace is a built in rule, I believe.  And there are various others tailored to different languages.

It’s used pretty extensively in Python development, but doesn’t need Python.  Since it runs in node, there’s a VSCode extension for it.

So, I was wondering if there was any interest in perhaps adding the yaml configurations to the wine source tree, and figuring out what rules might be useful to implement.

Thanks,
Rob