Hi, As part of this PR https://github.com/twisted/twisted/pull/1619 Thomas has requested to enable the https://pre-commit.ci service for twisted/twisted repository
The pre-commit.ci is kind of an auto-pilot for our pre-commit configuration. We already do post-commit checks via the pre-commit framework and I think that some of the core developers have pre-commit enabled as a git hook. The convenience if pre-commit.ci is that if you don't have the git-hook enable and you commit dirty code, pre-commit.ci will fix it for you. pre-commit.ci will also automatically create PR to update the pre-commit dependencies. This all comes at the cost of allowing the external pre-commit.ci server to write to twisted/twisted repository. pre-commit.ci was already enabled for twisted/ldaptor pre-commit.ci is currently enabled for twisted/twisted for testing while a final decision is made. I am +0 on this change due to security reasons...but I do think that it will reduce a bit of the frustration for first time contributors. If there are no strong comments against enabling pre-commit.ci, in a week, I plan to approve the PR and have the pre-commit.ci enabled for the long term. We can always rewoke to pre-commit.ci write access if needed. Cheers -- Adi Roiban
_______________________________________________ Twisted-Python mailing list Twisted-Python@twistedmatrix.com https://twistedmatrix.com/cgi-bin/mailman/listinfo/twisted-python
