Merge pull request #51 from George-Ogden/linter

Remove Linting Rule

Author: George-Ogden

.pre-commit-config.yaml

@@ -22,6 +22,12 @@ repos:
             symbols\ and\ spaces|
           )
 
+  - repo: https://github.com/George-Ogden/linter
+    rev: v1.1.0
+    hooks:
+      - id: lint
+        exclude: ^tests/[^/].*/test_data/
+
   - repo: https://github.com/astral-sh/ruff-pre-commit
     rev: v0.14.4
     hooks:
@@ -40,7 +46,6 @@ repos:
         exclude: ^tests/[^/]*/test_data/
         additional_dependencies:
           - GitPython
-          - libcst==1.8.5
           - pytest
 
   - repo: https://github.com/codespell-project/codespell

.pre-commit-hooks.yaml

@@ -43,10 +43,3 @@
   language: python
   stages: [commit-msg]
   additional_dependencies: [codespell]
-
-- id: dict-rewrite
-  name: Rewrite string-keyed dictionaries to use `dict`.
-  entry: dict-rewrite
-  types: [python]
-  language: python
-  additional_dependencies: [libcst==1.8.5]

README.md

@@ -18,7 +18,6 @@ This repository provides the following pre-commit hooks:
 - [check-merge-conflict](#check-merge-conflict) - check for merge conflicts.
 - [mypy](#mypy) - run [MyPy](#https://github.com/python/mypy).
 - [spell-check-commit-msgs](#spell-check-commit-msgs) - check for spelling errors in commit messages.
-- [dict-rewrite](#dict-rewrite) - rewrite string-keyed dictionaries to use `dict`.
 
 ### dbg-check
 
@@ -59,28 +58,6 @@ You can set the MyPy version in the requirements file (eg `mypy==1.17.1`), other
 This uses [`codespell`](https://github.com/codespell-project/codespell) under the hood, and accepts the same flags via the `args` field, but interactive mode is not supported.
 You need to ensure that you install the `commit-msg` hooks, which you can do with `pre-commit install -t pre-commit -t commit-msg` or adding `default_install_hook_types: ["pre-commit", "commit-msg"]` to the `.pre-commit-config.yaml` (like below).
 
-### dict-rewrite
-
-Make string-based dictionaries use `dict` instead of curly braces. For example,
-
-```python
-{
-  "a": "ascii",
-  "r": repr,
-  "integer": 15,
-}
-# is rewritten as
-dict(
-  a="ascii",
-  r=repr,
-  integer=15
-)
-```
-
-Use the `--fix` arg to apply the change, as well as linting files.
-The change may disrupt the style of the code, so consider using a formatter, such as [`ruff`](https://github.com/astral-sh/ruff/).
-It is possible to ignore dictionaries using `# dict-ignore`.
-
 ## Example Use
 
 Here's a sample `.pre-commit-config.yaml`:
@@ -97,8 +74,14 @@ repos:
       - id: mixed-line-ending
       - id: trailing-whitespace
 
+  - repo: https://github.com/George-Ogden/linter
+    rev: v1.1.0
+    hooks:
+      - id: lint
+        exclude: ^tests/[^/].*/test_data/
+
   - repo: https://github.com/George-Ogden/pre-commit-hooks/
-    rev: v1.4.3
+    rev: v2.0.0
     hooks:
       - id: dbg-check
         exclude: ^test/
@@ -109,8 +92,6 @@ repos:
       - id: mypy
         args: [-r, requirements.txt, --strict]
       - id: spell-check-commit-msgs
-      - id: dict-rewrite
-        args: [--fix]
 ```
 
 ### Development

dict_rewrite.py

@@ -1,18 +1,14 @@
 [project]
 name = "Pre-Commit-Hooks"
-version = "1.4.3"
+version = "2.0.0"
 requires-python = ">=3.9,<3.15"
 
-[project.scripts]
-dict-rewrite = "dict_rewrite:main_cli"
-
 [build-system]
 requires = ["setuptools"]
 build-backend = "setuptools.build_meta"
 
 [tool.setuptools]
 packages = []
-py-modules = ["dict_rewrite"]
 script-files = ["./run-spell-check.sh"]
 
 [tool.ruff]

pyproject.toml

@@ -1 +0,0 @@
-libcst==1.8.5