opendp · TedTed · Nov 17, 2025 · Nov 16, 2025
diff --git a/pyproject.toml b/pyproject.toml
@@ -229,7 +229,6 @@ ignore = [
     "RUF015",  # unnecessary-iterable-allocation-for-first-element
     "D417",    # undocumented-param
     "RUF007",  # zip-instead-of-pairwise
-    "D415",    # missing-terminal-punctuation
     "RUF043",  # pytest-raises-ambiguous-pattern
     "D205",    # missing-blank-line-after-summary
     "D210",    # surrounding-whitespace

diff --git a/test/unit/domains/test_base.py b/test/unit/domains/test_base.py
@@ -11,7 +11,7 @@
 
 
 class NewDomain(Domain):
-    """New Domain class that inherits from Domain"""
+    """New Domain class that inherits from Domain."""
 
     @property
     def carrier_type(self) -> type:

diff --git a/test/unit/domains/test_spark_domains.py b/test/unit/domains/test_spark_domains.py
@@ -1233,7 +1233,7 @@ class TestSparkColumnDescriptors:
 
     @pytest.fixture
     def test_df(self) -> DataFrame:
-        """Get a base DataFrame"""
+        """Get a base DataFrame."""
         return self.spark.createDataFrame(
             [
                 (

diff --git a/test/unit/measurements/test_interactive_measurements.py b/test/unit/measurements/test_interactive_measurements.py
@@ -1467,7 +1467,7 @@ def test_split(self):
             self.assertEqual(child.state, expected_state)
 
     def test_queue_transformation_on_active_accountant(self):
-        """queue_transformation runs immediately on active accountant"""
+        """queue_transformation runs immediately on active accountant."""
         accountant = PrivacyAccountant.launch(
             measurement=self.measurement, data=self.data
         )
@@ -1495,7 +1495,7 @@ def test_queue_transformation_on_active_accountant(self):
         self.assertIsNone(accountant._pending_transformation)
 
     def test_queue_transformation_on_inactive_accountant(self):
-        """queue_transformation queues transformations on inactive account"""
+        """queue_transformation queues transformations on inactive account."""
         accountant = PrivacyAccountant.launch(
             measurement=self.measurement, data=self.data
         )

diff --git a/test/unit/test_metrics.py b/test/unit/test_metrics.py
@@ -59,12 +59,12 @@ class TestNullMetric(TestCase):
     """TestCase for NullMetric."""
 
     def test_valid(self):
-        """Validate is not implemented"""
+        """Validate is not implemented."""
         with self.assertRaises(NotImplementedError):
             NullMetric().validate(3)
 
     def test_compare(self):
-        """Compare is not implemented"""
+        """Compare is not implemented."""
         with self.assertRaises(NotImplementedError):
             NullMetric().compare(3, 2)
 
@@ -1927,7 +1927,7 @@ def test_distance(self, metric: Metric, df1: Any, df2: Any, distance: Any):
 
 
 class TestDictMetric(TestCase):
-    """TestCase for DictMetric"""
+    """TestCase for DictMetric."""
 
     def test_constructor_mutable_arguments(self):
         """Tests that mutable constructor arguments are copied."""
@@ -2125,7 +2125,7 @@ def test_distance_empty(self):
 
 
 class TestAddRemoveKeys(PySparkTest):
-    """TestCase for AddRemoveKeys"""
+    """TestCase for AddRemoveKeys."""
 
     @parameterized.expand(
         [

diff --git a/test/unit/transformations/spark_transformations/test_join.py b/test/unit/transformations/spark_transformations/test_join.py
@@ -1370,7 +1370,7 @@ def test_columns_ordering(
     def test_stability_relation_and_function(
         self, d_in: ExactNumberInput, d_out: ExactNumberInput, expected: bool
     ):
-        """Test that PrivateJoinOnKey's stability relation and function are correct"""
+        """Test that PrivateJoinOnKey's stability relation and function are correct."""
         private_join = PrivateJoinOnKey(
             input_domain=DictDomain(
                 {"left": self.left_domain, "right": self.right_domain}

diff --git a/test/unit/utils/test_arb.py b/test/unit/utils/test_arb.py
@@ -1,4 +1,4 @@
-"""Test for :mod:`tmlt.core.utils.arb`"""
+"""Test for :mod:`tmlt.core.utils.arb`."""
 
 
 # SPDX-License-Identifier: Apache-2.0

diff --git a/test/unit/utils/test_cleanup.py b/test/unit/utils/test_cleanup.py
@@ -48,7 +48,7 @@ def setUp(self):
 
     @staticmethod
     def _recursive_remove(p: Path):
-        """Recursively remove a path (just like `rm -r`)"""
+        """Recursively remove a path (just like `rm -r`)."""
         if not p.is_dir():
             p.unlink()
         for f in p.iterdir():

diff --git a/test/unit/utils/test_misc.py b/test/unit/utils/test_misc.py
@@ -1,4 +1,4 @@
-"""Test for :mod:`tmlt.core.utils.misc`"""
+"""Test for :mod:`tmlt.core.utils.misc`."""
 
 # SPDX-License-Identifier: Apache-2.0
 # Copyright Tumult Labs 2025

diff --git a/test/unit/utils/test_testing.py b/test/unit/utils/test_testing.py
@@ -1,4 +1,4 @@
-"""Test for :mod:`tmlt.core.utils.testing`"""
+"""Test for :mod:`tmlt.core.utils.testing`."""
 
 # SPDX-License-Identifier: Apache-2.0
 # Copyright Tumult Labs 2025