-
Notifications
You must be signed in to change notification settings - Fork 526
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add clang-format check to CI #2816
Conversation
See CI error for unformatted code: https://github.com/llvm/torch-mlir/actions/runs/7674955115/job/20920465785?pr=2816 |
I don't have much experience with Github actions, so let me know if I'm doing something terribly wrong 🙂 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! I don't have any experience either though
Over the weekend, I switched us off of buildAndTest (to ci.yml). I created a dedicated lint pipeline. Should be trivial to add there: https://github.com/llvm/torch-mlir/blob/main/.github/workflows/lint.yml |
44cae2a
to
4561970
Compare
This PR adds a check to the CI right after checking out the Torch-MLIR repository to make sure that the changes in the PR don't require any `git clang-format` modifications.
4561970
to
b19e045
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks much
This PR adds a check to the CI right after checking out the Torch-MLIR repository to make sure that the changes in the PR don't require any
git clang-format
modifications.