Pre-commit configuration for Zarr-Python #1012
Replies: 4 comments 4 replies
-
Thanks for picking this up @Alt-Shivam. Looking forward to the PR. :) |
Beta Was this translation helpful? Give feedback.
-
👍 to having pre-commit on Zarr. It's really helpful. |
Beta Was this translation helpful? Give feedback.
-
So, For now, we're having flake8 in our workflow. #1015 So we've 3 choices now: |
Beta Was this translation helpful? Give feedback.
-
Hey folks, so should I create a PR that contains: Note: There are lots of files to be changed as per blake8 |
Beta Was this translation helpful? Give feedback.
-
Hey folks,
I reviewed this issue #978 for adding pre-commit configuration.
So to start with some initial configurations,
I am starting with this and this
Although @parthxtripathi has shown interest in this issue
But this issue is stale for the last 1 month, So I have also started working on this
will generate a PR with the initial config soon.
Till then, please post your suggestions on this Discussion.
cc @joshmoore @MSanKeys963
Beta Was this translation helpful? Give feedback.
All reactions