Mark abstract base classes and methods

Author: Avasam

newsfragments/4503.feature.rst

@@ -0,0 +1 @@
+Mark abstract base classes and methods with `abc.ABC` and `abc.abstract_method` -- by :user:`Avasam`

pkg_resources/__init__.py

@@ -22,6 +22,7 @@
 
 from __future__ import annotations
 
+from abc import ABC
 import sys
 
 if sys.version_info < (3, 8):  # noqa: UP036 # Check for unsupported versions
@@ -311,7 +312,7 @@ def get_supported_platform():
 ]
 
 
-class ResolutionError(Exception):
+class ResolutionError(Exception, ABC):
     """Abstract base for dependency resolution errors"""
 
     def __repr__(self):

setuptools/__init__.py

@@ -1,5 +1,6 @@
 """Extensions to the 'distutils' for large or complex distributions"""
 
+from abc import ABC, abstractmethod
 import functools
 import os
 import re
@@ -117,7 +118,7 @@ def setup(**attrs):
     _Command = monkey.get_unpatched(distutils.core.Command)
 
 
-class Command(_Command):
+class Command(_Command, ABC):
     """
     Setuptools internal actions are organized using a *command design pattern*.
     This means that each action (or group of closely related actions) executed during
@@ -226,6 +227,33 @@ def reinitialize_command(self, command, reinit_subcommands=False, **kw):
         vars(cmd).update(kw)
         return cmd
 
+    @abstractmethod
+    def initialize_options(self) -> None:
+        """
+        Set or (reset) all options/attributes/caches used by the command
+        to their default values. Note that these values may be overwritten during
+        the build.
+        """
+        raise NotImplementedError
+
+    @abstractmethod
+    def finalize_options(self) -> None:
+        """
+        Set final values for all options/attributes used by the command.
+        Most of the time, each option/attribute/cache should only be set if it does not
+        have any value yet (e.g. ``if self.attr is None: self.attr = val``).
+        """
+        raise NotImplementedError
+
+    @abstractmethod
+    def run(self) -> None:
+        """
+        Execute the actions intended by the command.
+        (Side effects **SHOULD** only take place when ``run`` is executed,
+        for example, creating new files or writing to the terminal output).
+        """
+        raise NotImplementedError
+
 
 def _find_all_simple(path):
     """

setuptools/command/setopt.py

@@ -1,3 +1,4 @@
+from abc import ABC, abstractmethod
 from distutils.util import convert_path
 from distutils import log
 from distutils.errors import DistutilsOptionError
@@ -68,7 +69,7 @@ def edit_config(filename, settings, dry_run=False):
             opts.write(f)
 
 
-class option_base(Command):
+class option_base(Command, ABC):
     """Abstract base class for commands that mess with config files"""
 
     user_options = [
@@ -103,6 +104,10 @@ def finalize_options(self):
             )
         (self.filename,) = filenames
 
+    @abstractmethod
+    def run(self) -> None:
+        raise NotImplementedError
+
 
 class setopt(option_base):
     """Save command-line options to a file"""

setuptools/sandbox.py

@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+from abc import ABC
 import os
 import sys
 import tempfile
@@ -265,7 +266,7 @@ def run_setup(setup_script, args):
             # Normal exit, just return
 
 
-class AbstractSandbox:
+class AbstractSandbox(ABC):
     """Wrap 'os' module and 'open()' builtin for virtualizing setup scripts"""
 
     _active = False