Revert "Backport PR #1626: (chore): automation of towncrier release process (…" (#1741)

This reverts commit 7d48a8a.

Author: ilan-gold

ci/scripts/towncrier_automation.py

@@ -6,21 +6,6 @@ AnnData follows the development practices outlined in the [Scanpy contribution g
 .. include:: _key_contributors.rst
 ```
 
-## Release Notes
-
-AnnData differs from `scanpy` (for now) in how its releases are done.
-It uses [towncrier][] to build its changelog.
-We have set up some automation around this process.
-To run `towncrier`, create a `PR` into the base branch of the release with the compiled changelog, and backport to `main` if needed (i.e., the base branch is something like `0.10.x`), run
-
-```shell
-hatch run towncrier:build X.Y.Z
-```
-
-You may add the option `--dry-run` at the end to do the local steps without pushing to Github, although the push will be mocked via [`gh pr --dry-run`](https://cli.github.com/manual/gh_pr_create).
-
-[towncrier]: https://towncrier.readthedocs.io/en/stable/
-
 ## CI
 
 ### GPU CI

docs/contributing.md

@@ -169,11 +169,9 @@ select = [
     "E",   # Error detected by Pycodestyle
     "F",   # Errors detected by Pyflakes
     "W",   # Warning detected by Pycodestyle
-    "PLW", # Pylint
     "UP",  # pyupgrade
     "I",   # isort
     "TCH", # manage type checking blocks
-    "TID", # Banned imports
     "ICN", # Follow import conventions
     "PTH", # Pathlib instead of os.path
     "PT",  # Pytest conventions
@@ -185,21 +183,13 @@ ignore = [
     "E731",
     # allow I, O, l as variable names -> I is the identity matrix, i, j, k, l is reasonable indexing notation
     "E741",
-    # We use relative imports from parent modules
-    "TID252",
-    # Shadowing loop variables isn’t a big deal
-    "PLW2901",
 ]
 [tool.ruff.lint.per-file-ignores]
 # E721 comparing types, but we specifically are checking that we aren't getting subtypes (views)
 "tests/test_readwrite.py" = ["E721"]
 [tool.ruff.lint.isort]
 known-first-party = ["anndata"]
 required-imports = ["from __future__ import annotations"]
-[tool.ruff.lint.flake8-tidy-imports.banned-api]
-"subprocess.call".msg = "Use `subprocess.run([…])` instead"
-"subprocess.check_call".msg = "Use `subprocess.run([…], check=True)` instead"
-"subprocess.check_output".msg = "Use `subprocess.run([…], check=True, capture_output=True)` instead"
 [tool.ruff.lint.flake8-type-checking]
 exempt-modules = []
 strict = true

pyproject.toml

@@ -31,7 +31,7 @@ def get_memory_usage():
         meminfo = process.get_memory_info()
     mem = meminfo[0] / 2**30  # output in GB
     mem_diff = mem
-    global _previous_memory_usage  # noqa: PLW0603
+    global _previous_memory_usage
     if _previous_memory_usage is not None:
         mem_diff = mem - _previous_memory_usage
     _previous_memory_usage = mem