bneradt commented on issue #11754: URL: https://github.com/apache/trafficserver/issues/11754#issuecomment-2329502105
Thanks for the observations. > Maybe consider using system yapf/clang-format and publishing config rather than version locking? We version lock these formatters to ensure that they format identically across all users. Different versions may behave differently with different configs. If something even very small changes about the way a tool formats, it gets in the way of a developer committing a patch because the different CI format will block it and there won't be much the dev can do. Even if they manually edit to satisfy CI, their own pre-commit hook will block them. So they will have to remove their pre-commit hook, which will then miss other formatting issues (maybe with clang-format) that won't get caught until CI runs. Having said that, we can look into updating yapf. I switched to yapf from autopep8 due to bugs in the recent version of autopep8 - with Python 3.12, autopep8 was simply broken. I actually contributed observations from ATS to the bug [here](https://github.com/hhatto/autopep8/issues/712). If I recall correctly (it's been about 10 months, so my recollection is hazy) I chose the latest version of yapf at the time that could easily be used across the Python versions we support. But maybe things are different now. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
