theochem/AtomDBdata

pre-commit CI will not work with this repo

Opened this issue · 5 comments

msricher commented

@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.

PaulWAyers commented

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.

msricher commented
PaulWAyers commented

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.

tovrstra commented

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.

  • 👍1
tovrstra commented

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.

  • 👍2