Update to modern packaging

.github/dependabot.yml

@@ -0,0 +1,17 @@
+# Please see the documentation for all configuration options:
+# https://docs.github.com/en/code-security/dependabot/dependabot-version-updates/configuration-options-for-the-dependabot.yml-file
+
+version: 2
+updates:
+  - package-ecosystem: "github-actions"
+    directory: "/"
+    schedule:
+      interval: "monthly"
+  - package-ecosystem: "pip"
+    directory: "/"
+    schedule:
+      interval: "monthly"
+    groups:
+      pip-packages:
+        patterns:
+          - "*"

.github/workflows/test.yml

@@ -7,21 +7,24 @@ jobs:
     name: Test on python ${{ matrix.python-version }} and ${{ matrix.os }}
     runs-on: ${{ matrix.os }}
     strategy:
+      fail-fast: false
       matrix:
         python-version: ["3.8", "3.9", "3.10"]
         os: [ubuntu-latest, windows-latest, macOS-latest]
 
     steps:
     - uses: actions/checkout@v4
-    - name: Set up Python ${{ matrix.python-version }}
-      uses: actions/setup-python@v5
+
+    - uses: astral-sh/setup-uv@v5
       with:
         python-version: ${{ matrix.python-version }}
-    - name: Install tox
-      run: |
-        python -m pip install --upgrade pip
-        pip install tox tox-gh-actions
-    - name: Test with tox
-      run: tox
+        # TODO(https://github.com/astral-sh/setup-uv/issues/226): Remove this.
+        prune-cache: ${{ matrix.os != 'windows-latest' }}
+
+    - run: uv sync --all-extras --dev
+
+    - run: uv pip install tox tox-gh-actions
+
+    - run: uv run tox
       env:
         PLATFORM: ${{ matrix.os }}

docs/Makefile

@@ -10,16 +10,16 @@ BUILDDIR      = _build
 
 # Put it first so that "make" without argument is like "make help".
 help:
-	@$(SPHINXBUILD) -M help "$(SOURCEDIR)" "$(BUILDDIR)" $(SPHINXOPTS) $(O)
+	uv run @$(SPHINXBUILD) -M help "$(SOURCEDIR)" "$(BUILDDIR)" $(SPHINXOPTS) $(O)
 
 .PHONY: help man Makefile
 
 # Copy generated manfile into project root for distribution.
 man: Makefile
-	@$(SPHINXBUILD) -M man "$(SOURCEDIR)" "$(BUILDDIR)" $(SPHINXOPTS) $(O)
+	uv run @$(SPHINXBUILD) -M man "$(SOURCEDIR)" "$(BUILDDIR)" $(SPHINXOPTS) $(O)
 	cp "$(BUILDDIR)/man/git-revise.1" "$(SOURCEDIR)/.."
 
 # Catch-all target: route all unknown targets to Sphinx using the new
 # "make mode" option.  $(O) is meant as a shortcut for $(SPHINXOPTS).
 %: Makefile
-	@$(SPHINXBUILD) -M $@ "$(SOURCEDIR)" "$(BUILDDIR)" $(SPHINXOPTS) $(O)
+	uv run @$(SPHINXBUILD) -M $@ "$(SOURCEDIR)" "$(BUILDDIR)" $(SPHINXOPTS) $(O)

docs/conf.py

@@ -15,6 +15,8 @@
 import os
 import sys
 
+from sphinx.application import Sphinx
+
 sys.path.insert(0, os.path.abspath(".."))
 
 import gitrevise
@@ -122,5 +124,5 @@
 # -- Extension configuration -------------------------------------------------
 
 
-def setup(app):
+def setup(app: Sphinx) -> None:
     app.add_object_type("gitconfig", "gitconfig", objname="git config value")

docs/contributing.rst

@@ -10,14 +10,14 @@ for testing, and :command:`black` for code formatting.
 
 .. code-block:: shell
 
-  $ tox           # All python versions
-  $ tox -e py38   # Python 3.8
-  $ tox -e py39   # Python 3.9
-  $ tox -e py310  # Python 3.10
+  $ uv run tox           # All python versions
+  $ uv run tox -e py38   # Python 3.8
+  $ uv run tox -e py39   # Python 3.9
+  $ uv run tox -e py310  # Python 3.10
 
-  $ tox -e mypy   # Mypy Typechecking
-  $ tox -e lint   # Linting
-  $ tox -e format # Check Formatting
+  $ uv run tox -e mypy   # Mypy Typechecking
+  $ uv run tox -e lint   # Linting
+  $ uv run tox -e format # Check Formatting
 
 Code Formatting
 ---------------
@@ -26,8 +26,8 @@ This project uses ``isort`` and ``black`` for code formatting.
 
 .. code-block:: shell
 
-  $ isort .  # sort imports
-  $ black .  # format all python code
+  $ uv run isort .     # sort imports
+  $ uv run ruff format # format all python code
 
 Building Documentation
 ----------------------
@@ -44,6 +44,6 @@ Publishing
 
 .. code-block:: shell
 
-  $ python3 setup.py sdist bdist_wheel
-  $ twine check dist/*
-  $ twine upload dist/*
+  $ uv build
+  $ uv run twine check dist/*
+  $ uv run twine upload dist/*

gitrevise/odb.py

@@ -31,6 +31,8 @@
 if TYPE_CHECKING:
     from subprocess import _FILE
 
+    from typing_extensions import Self
+
 
 class MissingObject(Exception):
     """Exception raised when a commit cannot be found in the ODB"""
@@ -57,7 +59,7 @@ class Oid(bytes):
     def __new__(cls, b: bytes) -> Oid:
         if len(b) != 20:
             raise ValueError("Expected 160-bit SHA1 hash")
-        return super().__new__(cls, b)  # type: ignore
+        return super().__new__(cls, b)
 
     @classmethod
     def fromhex(cls, instr: str) -> Oid:
@@ -165,8 +167,8 @@ class Repository:
     """path to GnuPG binary"""
 
     _objects: Dict[int, Dict[Oid, GitObj]]
-    _catfile: Popen
-    _tempdir: Optional[TemporaryDirectory]
+    _catfile: Popen[bytes]
+    _tempdir: Optional[TemporaryDirectory[str]]
 
     __slots__ = [
         "workdir",
@@ -480,7 +482,7 @@ class GitObj:
 
     __slots__ = ("repo", "body", "oid", "persisted")
 
-    def __new__(cls: Type[GitObjT], repo: Repository, body: bytes) -> GitObjT:
+    def __new__(cls, repo: Repository, body: bytes) -> "Self":
         oid = Oid.for_object(cls._git_type(), body)
         cache = repo._objects[oid[0]]  # pylint: disable=protected-access
         if oid in cache:
@@ -495,7 +497,7 @@ def __new__(cls: Type[GitObjT], repo: Repository, body: bytes) -> GitObjT:
         self.persisted = False
         cache[oid] = self
         self._parse_body()  # pylint: disable=protected-access
-        return cast(GitObjT, self)
+        return self
 
     @classmethod
     def _git_type(cls) -> str:
@@ -834,7 +836,7 @@ def git(
         trim_newline: bool = True,
     ) -> bytes:
         """Invoke git with the given index as active"""
-        env = dict(**env) if env is not None else dict(**os.environ)
+        env = {**env} if env is not None else {**os.environ}
         env["GIT_INDEX_FILE"] = str(self.index_file)
         return self.repo.git(
             *cmd,

gitrevise/tui.py

@@ -188,7 +188,7 @@ def noninteractive(
 
     # Update the commit message on the target commit if requested.
     if args.message:
-        message = b"\n".join(l.encode("utf-8") + b"\n" for l in args.message)
+        message = b"\n".join(line.encode("utf-8") + b"\n" for line in args.message)
         current = current.update(message=message)
 
     # Prompt the user to edit the commit message if requested.

pyproject.toml

@@ -0,0 +1,92 @@
+[build-system]
+requires = ["hatchling"]
+build-backend = "hatchling.build"
+
+[project]
+name = "git-revise"
+dynamic = ["version"]
+requires-python = ">=3.8"
+authors = [{ name = "Nika Layzell", email = "[email protected]" }]
+description = "Efficiently update, split, and rearrange git commits"
+readme = "README.md"
+license = { file = "LICENSE" }
+keywords = ["git", "revise", "rebase", "amend", "fixup"]
+classifiers = [
+  "Development Status :: 4 - Beta",
+  "Intended Audience :: Developers",
+  "Environment :: Console",
+  "Topic :: Software Development :: Version Control",
+  "Topic :: Software Development :: Version Control :: Git",
+  "License :: OSI Approved :: MIT License",
+  "Programming Language :: Python :: 3",
+  "Programming Language :: Python :: 3.8",
+  "Programming Language :: Python :: 3.9",
+  "Programming Language :: Python :: 3.10",
+]
+
+[project.scripts]
+git-revise = "gitrevise.tui:main"
+
+[dependency-groups]
+dev = [
+  "isort~=5.13.2",
+  "mypy~=1.14.1",
+  "pylint~=3.2.7",
+  "pytest-xdist~=3.6.1",
+  "pytest~=8.3.4",
+  "ruff~=0.9.6",
+  "sphinx~=7.1.2",
+  "tox-uv~=1.13.1",
+  "tox~=4.24.1",
+  "twine~=6.1.0",
+  "typing-extensions~=4.12.2",
+]
+
+[project.urls]
+Homepage = "https://github.com/mystor/git-revise/"
+Issues = "https://github.com/mystor/git-revise/issues/"
+Repository = "https://github.com/mystor/git-revise/"
+Documentation = "https://git-revise.readthedocs.io/en/latest/"
+
+[tool.hatch.version]
+path = "gitrevise/__init__.py"
+
+[tool.hatch.build.targets.wheel]
+packages = ["/gitrevise"]
+
+[tool.hatch.build.targets.wheel.shared-data]
+"git-revise.1" = "share/man/man1/git-revise.1"
+
+[tool.hatch.build.targets.sdist]
+include = ["/gitrevise"]
+
+[tool.pylint.messages_control]
+disable = [
+  "missing-docstring",
+  "too-few-public-methods",
+  "too-many-arguments",
+  "too-many-branches",
+  "too-many-instance-attributes",
+  "too-many-return-statements",
+  "cyclic-import",
+  "fixme",
+
+  # Currently broken analyses which are also handled (better) by mypy
+  "class-variable-slots-conflict",
+  "no-member",
+]
+
+good-names = [
+  # "Exception as e" is perfectly fine.
+  "e",
+  # "with open(…) as f" is idiomatic.
+  "f",
+  # Other contextually-unambiguous names.
+  "fn",
+  "repo",
+  "ed",
+]
+
+# TODO(https://github.com/astral-sh/ruff/issues/14813): Remove this once ruff properly respects
+# requires-python.
+[tool.ruff]