deprecate check-docstring-first without replacement this hook has unfixable false positives and is not that useful perhaps someone can write a flake8 plugin to replace the functionality
diff --git a/.pre-commit-hooks.yaml b/.pre-commit-hooks.yaml index a98a1e5..275605e 100644 --- a/.pre-commit-hooks.yaml +++ b/.pre-commit-hooks.yaml
@@ -29,7 +29,7 @@ entry: check-case-conflict language: python - id: check-docstring-first - name: check docstring is first + name: check docstring is first (deprecated) description: checks a common error of defining a docstring after code. entry: check-docstring-first language: python
diff --git a/README.md b/README.md index 9ee1677..8432455 100644 --- a/README.md +++ b/README.md
@@ -45,9 +45,6 @@ #### `check-case-conflict` Check for files with names that would conflict on a case-insensitive filesystem like MacOS HFS+ or Windows FAT. -#### `check-docstring-first` -Checks for a common error of placing code before the docstring. - #### `check-executables-have-shebangs` Checks that non-binary executables have a proper shebang. @@ -207,6 +204,8 @@ - `check-byte-order-marker`: instead use fix-byte-order-marker - `fix-encoding-pragma`: instead use [`pyupgrade`](https://github.com/asottile/pyupgrade) +- `check-docstring-first`: fundamentally flawed, deprecated without replacement. + ### As a standalone package