-
Notifications
You must be signed in to change notification settings - Fork 15
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
Use pre-commit action against merge-base #33
Conversation
Caching works just fine with the action! In the first passing run, pre-commit required 55s, in the run after that only 20s 🎉 |
@hofbi let's have pre-commit integration even nicer, as e.g. at CLIUtils/CLI11#689. Can you authorize github at https://pre-commit.ci/ and add pre-commit ci to this repo? |
Ok, done. To trigger the CI you could also update https://github.com/hofbi/telecarla/pull/33/files#diff-b803fcb7f17ed9235f1e5cb1fcd2f5d3b2838429d4368ae4c57ce4436577f03fR15 to |
Badge done, what exactly do you mean with the |
This reverts commit 4be0a95.
Is this one ready? |
Yes, except the v2 thing that I didn't understand.Is that resolved? |
Resolve https://github.com/hofbi/telecarla/pull/32/files#r803870102 and https://github.com/hofbi/telecarla/pull/32/files#r804069423