DX: switch to issue forms

https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/syntax-for-issue-forms

.vscode/settings.json

@@ -56,8 +56,6 @@
     ".constraints/*.txt": true,
     ".github/workflows/cd.yml": true,
     ".github/workflows/ci.yml": true,
-    "src/repoma/.github/ISSUE_TEMPLATE": true,
-    "src/repoma/.github/pull_request_template.md": true,
     "src/repoma/.github/workflows/clean-caches.yml": true,
     "src/repoma/.github/workflows/pr-linting.yml": true,
     "src/repoma/.github/workflows/release-drafter.yml": true,

src/repoma/check_dev_files/__init__.py

@@ -14,7 +14,6 @@
     cspell,
     editorconfig,
     github_labels,
-    github_templates,
     github_workflows,
     gitpod,
     mypy,
@@ -65,6 +64,12 @@ def main(argv: Optional[Sequence[str]] = None) -> int:
         required=False,
         type=str,
     )
+    parser.add_argument(
+        "--keep-issue-templates",
+        help="Do not remove the .github/ISSUE_TEMPLATE directory",
+        action="store_true",
+        default=False,
+    )
     parser.add_argument(
         "--ignore-author",
         action="store_true",
@@ -165,7 +170,6 @@ def main(argv: Optional[Sequence[str]] = None) -> int:
     executor(editorconfig.main, args.no_python)
     if not args.allow_labels:
         executor(github_labels.main)
-    executor(github_templates.main)
     executor(
         github_workflows.main,
         allow_deprecated=args.allow_deprecated_workflows,
@@ -194,7 +198,7 @@ def main(argv: Optional[Sequence[str]] = None) -> int:
         executor(pyupgrade.main)
         executor(ruff.main)
         executor(setup_cfg.main, args.ignore_author)
-    executor(remove_deprecated_tools)
+    executor(remove_deprecated_tools, args.keep_issue_templates)
     executor(vscode.main)
     if not args.no_gitpod:
         executor(gitpod.main)

src/repoma/check_dev_files/deprecated.py

@@ -1,5 +1,6 @@
 """Remove deprecated linters and formatters."""
 import os
+import shutil
 from typing import TYPE_CHECKING, List
 
 from repoma.errors import PrecommitError
@@ -18,10 +19,12 @@
     from tomlkit.items import Table
 
 
-def remove_deprecated_tools() -> None:
+def remove_deprecated_tools(keep_issue_templates: bool) -> None:
     executor = Executor()
     executor(_remove_flake8)
     executor(_remove_isort)
+    if not keep_issue_templates:
+        executor(_remove_github_issue_templates)
     executor(_remove_pydocstyle)
     executor(_remove_pylint)
     executor.finalize()
@@ -85,6 +88,15 @@ def __remove_nbqa_option(option: str) -> None:
     raise PrecommitError(msg)
 
 
+def _remove_github_issue_templates() -> None:
+    __remove_configs(
+        [
+            ".github/ISSUE_TEMPLATE",
+            ".github/pull_request_template.md",
+        ]
+    )
+
+
 def _remove_pydocstyle() -> None:
     executor = Executor()
     executor(
@@ -122,7 +134,10 @@ def __remove_configs(paths: List[str]) -> None:
 def __remove_file(path: str) -> None:
     if not os.path.exists(path):
         return
-    os.remove(path)
+    if os.path.isdir(path):
+        shutil.rmtree(path)
+    else:
+        os.remove(path)
     msg = f"Removed {path}"
     raise PrecommitError(msg)