You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
is about a minimal configuration if the idea here is to add hooks that are agnostic to most other choices then I could make a PR to https://github.com/scikit-hep/cookie instead (if people think this is a good idea in either place I'll happily make a PR to wherever).
The text was updated successfully, but these errors were encountered:
Maybe it would be better as a mention (+ cookie)? Though this is really not meant to be minimal, so I think I'd be okay with it in the standard set. There may be cases where this is customized, but those are likely existing packages & not good practice for a new package.
As we advocate for the use of
pytest
, the https://github.com/pre-commit/pre-commit-hooks hookname-tests-test
might be a useful hook to add to the suggested hooks. Though asscikit-hep.github.io/pages/developers/style.md
Lines 24 to 41 in 994255a
is about a minimal configuration if the idea here is to add hooks that are agnostic to most other choices then I could make a PR to https://github.com/scikit-hep/cookie instead (if people think this is a good idea in either place I'll happily make a PR to wherever).
The text was updated successfully, but these errors were encountered: