Skip to content

Commit b467680

Browse files
bryevdvtcmetzger
andauthored
Remove black format pre-commit (#11)
* Remove black format pre-commit cc @ianthomas23 @pavithraes @tcmetzger Submitted for consideration. I am dubious of `black` formatting in general these days, and especially in notebooks. But I think as @ianthomas23 has pointed out we should not sacrifice accepted mathematical notation just to appease a formatter. * update readme --------- Co-authored-by: tcmetzger <[email protected]>
1 parent e6c61ba commit b467680

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

.pre-commit-config.yaml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,5 @@ repos:
22
- repo: https://github.com/nbQA-dev/nbQA
33
rev: 1.5.3
44
hooks:
5-
- id: nbqa-black
6-
args: [--diff, --line-length=256]
7-
additional_dependencies: [black==22.10.0]
85
- id: nbqa-flake8
96
additional_dependencies: [flake8==6.0.0]

README.md

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -99,8 +99,11 @@ environment):
9999
pre-commit install
100100
```
101101

102-
This way, some basic linting and formatting checks will be run on your code before
103-
you commit it.
102+
This way, some basic linting will be run on your code before you commit it.
103+
104+
For consistency, we ask that you generally follow the
105+
[Black code style](https://black.readthedocs.io/en/stable/the_black_code_style/current_style.html)
106+
wherever possible.
104107

105108
### Making changes
106109

0 commit comments

Comments
 (0)