-
Notifications
You must be signed in to change notification settings - Fork 173
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Support for pre-commit project #692
Comments
@jvstein Thanks for sharing your work. pre-commit looks like a nice tool. I've used Husky for the same purpose on previous projects. I've been thinking that one of the bits of missing documentation for dotnet-format is how to integrate with various build systems. There have been other issues opened where people have contributed their GitHub Action or AzDO yaml configurations. What might be nice is to start a new document ( We would be thrilled if you wanted to start the document and share your configuration for pre-commit with a little write-up. |
I totally endorse this. I think we start with a doc that we can point people at the tells them how to set this up and see where things go from there. |
That sounds like a great place to start. I'm happy to get that doc started. |
pre-commit has just added support for the dotnet language, so I've opened a PR to add a hooks file #872. |
I wrote a hook for integrating with the pre-commit project. It's here: https://github.com/jvstein/pre-commit-dotnet-format
I've been using it while developing on Linux, but haven't tested it on Windows or macOS. It's very much dependent on Docker, using a prebuilt docker image, because pre-commit doesn't support C# yet. I'd be very happy to integrate it into the project, but wasn't sure how to go about adding the docker dependency to the build system.
The text was updated successfully, but these errors were encountered: