pre-commit.ci

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

pre-commit.ci

Michael A. Smith
I noticed that the proposed rust PR (
https://github.com/apache/avro/pull/1234) has a .pre-commit-config.yaml
file. pre-commit.ci is a service that will automatically apply your
pre-commit hooks across your entire repo, and it's free for open source
projects.

Is this something we'd be interested in turning on for all of apache/avro?
(We might need infra-* help to fully enable it.)

I made a PR in my fork that shows what it would do if we turned it on.
- https://github.com/kojiromike/avro/pull/86

It's pretty configurable, so it certainly doesn't have to do everything I
have it doing in my example. This email is more about "if we're going to
have a .pre-commit-config.yaml, do we want to auto-apply it across all the
implementations?" than specifically what checks we should have.