Merge pull request #1287 from linsword13/cleanup

douglasjacobsen · web-flow · commit 55d9cd009f19 · 2025-10-21T19:47:50.000-06:00
Remove unused noqa comments
diff --git a/lib/ramble/ramble/schema/applications.py b/lib/ramble/ramble/schema/applications.py
@@ -10,7 +10,7 @@
 
 .. literalinclude:: _ramble_root/lib/ramble/ramble/schema/applications.py
    :lines: 12-
-"""  # noqa E501
+"""
 
 from llnl.util.lang import union_dicts
 
diff --git a/lib/ramble/ramble/schema/env_vars.py b/lib/ramble/ramble/schema/env_vars.py
@@ -10,7 +10,7 @@
 
 .. literalinclude:: _ramble_root/lib/ramble/ramble/schema/env_vars.py
    :lines: 12-
-"""  # noqa E501
+"""
 
 import ramble.schema.licenses
 
diff --git a/lib/ramble/ramble/schema/formatted_executables.py b/lib/ramble/ramble/schema/formatted_executables.py
@@ -10,7 +10,7 @@
 
 .. literalinclude:: _ramble_root/lib/ramble/ramble/schema/formatted_executables.py
    :lines: 12-
-"""  # noqa E501
+"""
 
 #: Properties for inclusion in other schemas
 properties = {
diff --git a/lib/ramble/ramble/schema/internals.py b/lib/ramble/ramble/schema/internals.py
@@ -10,7 +10,7 @@
 
 .. literalinclude:: _ramble_root/lib/ramble/ramble/schema/internals.py
    :lines: 12-
-"""  # noqa E501
+"""
 
 from llnl.util.lang import union_dicts
 
diff --git a/lib/ramble/ramble/schema/licenses.py b/lib/ramble/ramble/schema/licenses.py
@@ -10,7 +10,7 @@
 
 .. literalinclude:: _ramble_root/lib/ramble/ramble/schema/licenses.py
    :lines: 12-
-"""  # noqa E501
+"""
 
 import spack.schema.environment
 
diff --git a/lib/ramble/ramble/schema/modifiers.py b/lib/ramble/ramble/schema/modifiers.py
@@ -10,7 +10,7 @@
 
 .. literalinclude:: _ramble_root/lib/ramble/ramble/schema/modifiers.py
    :lines: 12-
-"""  # noqa E501
+"""
 
 section_name = "modifiers"
 
diff --git a/lib/ramble/ramble/schema/software.py b/lib/ramble/ramble/schema/software.py
@@ -10,7 +10,7 @@
 
 .. literalinclude:: _ramble_root/lib/ramble/ramble/schema/software.py
    :lines: 12-
-"""  # noqa E501
+"""
 
 import ramble.namespace
 
diff --git a/lib/ramble/ramble/schema/success_criteria.py b/lib/ramble/ramble/schema/success_criteria.py
@@ -10,7 +10,7 @@
 
 .. literalinclude:: _ramble_root/lib/ramble/ramble/schema/success_criteria.py
    :lines: 12-
-"""  # noqa E501
+"""
 
 
 success_criteria_def = {
diff --git a/lib/ramble/ramble/schema/variables.py b/lib/ramble/ramble/schema/variables.py
@@ -10,7 +10,7 @@
 
 .. literalinclude:: _ramble_root/lib/ramble/ramble/schema/variables.py
    :lines: 12-
-"""  # noqa E501
+"""
 
 variables_def = {
     "type": ["object", "null"],
diff --git a/lib/ramble/ramble/schema/variants.py b/lib/ramble/ramble/schema/variants.py
@@ -10,7 +10,7 @@
 
 .. literalinclude:: _ramble_root/lib/ramble/ramble/schema/variants.py
    :lines: 12-
-"""  # noqa E501
+"""
 
 variants_def = {
     "type": ["object", "null"],
diff --git a/lib/ramble/ramble/schema/workspace.py b/lib/ramble/ramble/schema/workspace.py
@@ -10,7 +10,7 @@
 
 .. literalinclude:: _ramble_root/lib/ramble/ramble/schema/workspace.py
    :lines: 12-
-"""  # noqa E501
+"""
 
 from llnl.util.lang import union_dicts
 
diff --git a/lib/ramble/ramble/schema/zips.py b/lib/ramble/ramble/schema/zips.py
@@ -10,7 +10,7 @@
 
 .. literalinclude:: _ramble_root/lib/ramble/ramble/schema/zips.py
    :lines: 12-
-"""  # noqa E501
+"""
 
 
 zip_def = {"type": "array", "default": [], "items": {"type": "string"}}
diff --git a/lib/ramble/ramble/test/end_to_end/experiment_excludes.py b/lib/ramble/ramble/test/end_to_end/experiment_excludes.py
@@ -174,7 +174,7 @@ def test_wrfv4_exclusions(mutable_config, mutable_mock_workspace_path, workspace
         assert search_files_for_string(
             out_files,
             "Would download https://www2.mmm.ucar.edu/wrf/users/benchmark/v422/v42_bench_conus12km.tar.gz",
-        )  # noqa
+        )
 
         # Test software directories
         software_dirs = ["wrfv4", "wrfv4-portable"]
diff --git a/lib/ramble/ramble/test/end_to_end/experiment_repeats.py b/lib/ramble/ramble/test/end_to_end/experiment_repeats.py
@@ -107,7 +107,7 @@ def test_gromacs_repeats(mutable_config, mutable_mock_workspace_path, workspace_
         assert search_files_for_string(
             out_files,
             "Would download https://ftp.gromacs.org/pub/benchmarks/water_GMX50_bare.tar.gz",
-        )  # noqa
+        )
 
         # Test software directories
         software_dirs = ["gromacs"]
diff --git a/lib/ramble/ramble/test/end_to_end/explicit_zips.py b/lib/ramble/ramble/test/end_to_end/explicit_zips.py
@@ -150,7 +150,7 @@ def test_wrfv4_explicit_zips(mutable_config, mutable_mock_workspace_path, worksp
         assert search_files_for_string(
             out_files,
             "Would download https://www2.mmm.ucar.edu/wrf/users/benchmark/v422/v42_bench_conus12km.tar.gz",
-        )  # noqa
+        )
 
         # Test software directories
         software_dirs = ["wrfv4", "wrfv4-portable"]
diff --git a/lib/ramble/ramble/test/end_to_end/included_configuration_files.py b/lib/ramble/ramble/test/end_to_end/included_configuration_files.py
@@ -138,7 +138,7 @@ def test_included_configuration_files(mutable_config, mutable_mock_workspace_pat
         assert search_files_for_string(
             out_files,
             "Would download https://www2.mmm.ucar.edu/wrf/users/benchmark/v422/v42_bench_conus12km.tar.gz",
-        )  # noqa
+        )
 
         # Test software directories
         software_dirs = ["wrfv4", "wrfv4-portable"]
diff --git a/lib/ramble/ramble/test/end_to_end/wrfv4_dry_run.py b/lib/ramble/ramble/test/end_to_end/wrfv4_dry_run.py
@@ -160,7 +160,7 @@ def test_wrfv4_spack_dry_run(mutable_config, mutable_mock_workspace_path, worksp
         assert search_files_for_string(
             out_files,
             "Would download https://www2.mmm.ucar.edu/wrf/users/benchmark/v422/v42_bench_conus12km.tar.gz",
-        )  # noqa
+        )
 
         # Test software directories
         software_dirs = ["wrfv4", "wrfv4-portable"]
@@ -409,7 +409,7 @@ def test_wrfv4_no_pkg_man_dry_run(mutable_config, mutable_mock_workspace_path, w
         assert search_files_for_string(
             out_files,
             "Would download https://www2.mmm.ucar.edu/wrf/users/benchmark/v422/v42_bench_conus12km.tar.gz",
-        )  # noqa
+        )
 
         expected_experiments = [
             "scaling_1_part1_wrfv4",
diff --git a/lib/ramble/ramble/util/file_cache.py b/lib/ramble/ramble/util/file_cache.py
@@ -125,7 +125,7 @@ def write_transaction(self, key):
         # TODO: the locking code.
         class WriteContextManager:
 
-            def __enter__(cm):  # noqa
+            def __enter__(cm):
                 cm.orig_filename = self.cache_path(key)
                 cm.orig_file = None
                 if os.path.exists(cm.orig_filename):
@@ -136,7 +136,7 @@ def __enter__(cm):  # noqa
 
                 return cm.orig_file, cm.tmp_file
 
-            def __exit__(cm, type, value, traceback):  # noqa
+            def __exit__(cm, type, value, traceback):
                 if cm.orig_file:
                     cm.orig_file.close()
                 cm.tmp_file.close()
