-
Notifications
You must be signed in to change notification settings - Fork 2
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
pre-commit CI will not work with this repo #8
Comments
You're admin for this repository @msricher . Is that not enough to control this? To me, since there is no precommit yaml in this root directory, I would not have expected precommit to be used here. |
There's no setting to control it that I can find. It's enabled at the
organization level, I think.
…On Fri., May 10, 2024, 12:48 Paul W. Ayers ***@***.***> wrote:
You're admin for this repository @msricher <https://github.com/msricher>
. Is that not enough to control this?
To me, since there is no precommit yaml in this root directory, I would
not have expected precommit to be used here.
—
Reply to this email directly, view it on GitHub
<#8 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADLODOT642FFNWNLWYP4E7LZBT27LAVCNFSM6AAAAABHNDLLHGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBUHEZDINBWGY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
I don't see an organization-level setting. If you can find the documentation for it, send it to me and I'll see if I can grok it. I think that @tovrstra is the main local expert on precommit hooks. |
The settings of external GitHub Apps can be found here: https://github.com/organizations/theochem/settings/installations Until now, we had pre-commit enabled on all repositories. I've switched this to "selected repositories" and enabled it for IOData and denspart. If more need to be added, feel free to select some more. |
This may also be interesting: https://github.com/pre-commit/action With this GitHub action you can also run pre-commit. It will be slower than pre-commit.ci, but it may have less restrictions in terms of repository size etc. |
@FarnazH pre-commit will not work with this repo because it's too large for pre-commit to clone it on its service. It's not required on this repo anyway because it just hosts data. Is it possible to disable pre-commit CI on this repo? @PaulWAyers didn't seem to know if it was possible, so I'm reaching out to you. I don't have admin on theochem.
The text was updated successfully, but these errors were encountered: