chore(pre-commit): remove check-docstring-first

Remove check-docstring-first pre-commit hook as it doesn't support
using Sphinx attribute docstrings. Attribute docstrings, as defined
in PEP-257 are strings immediately following top level module names.

The check-docstring-first hook incorrectly identifies this attribute
docstring as a module docstring, indicating either:

- "Multiple module docstrings (first docstring on line N)", or
- "Module docstring appears after code (code seen on line N)"

For details see related discussion on pre-commit issue:

    pre-commit/pre-commit-hooks#159

Signed-off-by: Stephan Linz <[email protected]>

Author: rexut

.pre-commit-config.yaml

@@ -19,7 +19,9 @@ repos:
       - id: debug-statements
       - id: check-builtin-literals
       - id: check-case-conflict
-      - id: check-docstring-first
+      # Disable the check-docstring-first hook, can be re-enabled when
+      # https://github.com/pre-commit/pre-commit-hooks/issues/159 is fixed.
+      # id: check-docstring-first
       - id: check-json
       - id: check-toml
       - id: check-xml