fix linting

Author: jnnsbrr

pycoupler/release.py

@@ -267,7 +267,7 @@ def main():
                 print(f"  - Major increment: {major + 1}.0.0")
         else:
             print(
-                "No current version found. Any valid semantic version (X.Y.Z) is allowed."
+                "No current version found. Any valid semantic version (X.Y.Z) is allowed."  # noqa: E501
             )
         print("")
         print("Examples of valid semantic versioning:")
@@ -276,18 +276,24 @@ def main():
         print("  - 1.0.0 -> 2.0.0 (major)")
         print("  - 1.0.0 -> 1.0.0 (re-release)")
         sys.exit(1)
-    
+
     # Check if this is a re-release and ask for confirmation
-    if current_version and parse_version(current_version) == parse_version(version):
-        print(f"⚠️  WARNING: You are attempting to re-release version {version}")
-        print(f"This will delete the existing tag and create a new one with recent changes.")
+    if current_version and parse_version(current_version) == parse_version(version):  # noqa: E501
+        print(f"⚠️  WARNING: You are attempting to re-release version {version}")  # noqa: E501
+        print(
+            f"This will delete the existing tag and create a new one with recent changes."  # noqa: E501
+        )
         print("")
         while True:
-            response = input("Are you certain you want to re-release this version? [y/N]: ").strip().lower()
-            if response in ['y', 'yes']:
+            response = (
+                input("Are you certain you want to re-release this version? [y/N]: ")  # noqa: E501
+                .strip()
+                .lower()
+            )
+            if response in ["y", "yes"]:
                 print("✅ Re-release confirmed. Proceeding...")
                 break
-            elif response in ['n', 'no', '']:
+            elif response in ["n", "no", ""]:
                 print("❌ Re-release cancelled.")
                 sys.exit(0)
             else:

tests/test_release.py

@@ -443,16 +443,12 @@ def test_get_current_version(self):
 
         # Test with valid tag
         with patch("subprocess.run") as mock_run:
-            mock_run.return_value = MagicMock(
-                stdout="v1.5.0\n", returncode=0
-            )
+            mock_run.return_value = MagicMock(stdout="v1.5.0\n", returncode=0)
             assert get_current_version() == "1.5.0"
 
         # Test with tag without 'v' prefix
         with patch("subprocess.run") as mock_run:
-            mock_run.return_value = MagicMock(
-                stdout="1.5.0\n", returncode=0
-            )
+            mock_run.return_value = MagicMock(stdout="1.5.0\n", returncode=0)
             assert get_current_version() == "1.5.0"
 
         # Test with no tags
@@ -470,9 +466,15 @@ class TestReReleaseConfirmation:
     @patch("pycoupler.release.get_current_branch")
     @patch("pycoupler.release.get_current_version")
     @patch("subprocess.run")
-    def test_release_confirmation_yes(self, mock_subprocess, mock_current_version, 
-                                    mock_branch, mock_package, mock_update_citation, 
-                                    mock_run_command):
+    def test_release_confirmation_yes(
+        self,
+        mock_subprocess,
+        mock_current_version,
+        mock_branch,
+        mock_package,
+        mock_update_citation,
+        mock_run_command,
+    ):
         """Test re-release confirmation with 'yes' response."""
         # Setup mocks
         mock_package.return_value = "test-package"
@@ -497,9 +499,15 @@ def test_release_confirmation_yes(self, mock_subprocess, mock_current_version,
     @patch("pycoupler.release.get_current_branch")
     @patch("pycoupler.release.get_current_version")
     @patch("subprocess.run")
-    def test_release_confirmation_no(self, mock_subprocess, mock_current_version, 
-                                   mock_branch, mock_package, mock_update_citation, 
-                                   mock_run_command):
+    def test_release_confirmation_no(
+        self,
+        mock_subprocess,
+        mock_current_version,
+        mock_branch,
+        mock_package,
+        mock_update_citation,
+        mock_run_command,
+    ):
         """Test re-release confirmation with 'no' response."""
         # Setup mocks
         mock_package.return_value = "test-package"
@@ -523,9 +531,15 @@ def test_release_confirmation_no(self, mock_subprocess, mock_current_version,
     @patch("pycoupler.release.get_current_branch")
     @patch("pycoupler.release.get_current_version")
     @patch("subprocess.run")
-    def test_release_confirmation_invalid_then_valid(self, mock_subprocess, mock_current_version, 
-                                                    mock_branch, mock_package, mock_update_citation, 
-                                                    mock_run_command):
+    def test_release_confirmation_invalid_then_valid(
+        self,
+        mock_subprocess,
+        mock_current_version,
+        mock_branch,
+        mock_package,
+        mock_update_citation,
+        mock_run_command,
+    ):
         """Test re-release confirmation with invalid then valid response."""
         # Setup mocks
         mock_package.return_value = "test-package"
@@ -550,9 +564,15 @@ def test_release_confirmation_invalid_then_valid(self, mock_subprocess, mock_cur
     @patch("pycoupler.release.get_current_branch")
     @patch("pycoupler.release.get_current_version")
     @patch("subprocess.run")
-    def test_no_confirmation_for_new_version(self, mock_subprocess, mock_current_version, 
-                                            mock_branch, mock_package, mock_update_citation, 
-                                            mock_run_command):
+    def test_no_confirmation_for_new_version(
+        self,
+        mock_subprocess,
+        mock_current_version,
+        mock_branch,
+        mock_package,
+        mock_update_citation,
+        mock_run_command,
+    ):
         """Test that no confirmation is needed for new versions."""
         # Setup mocks
         mock_package.return_value = "test-package"
@@ -612,7 +632,9 @@ def test_full_release_workflow_mock(self):
 
                     mock_run_command.return_value = True
                     mock_subprocess.return_value = MagicMock(returncode=0)
-                    mock_current_version.return_value = "1.0.0"  # Valid previous version
+                    mock_current_version.return_value = (
+                        "1.0.0"  # Valid previous version
+                    )
 
                     # Test the main function
                     with patch.object(sys, "argv", ["release.py", "2.0.0"]):