fix: address review feedback

Author: guptakushal03

crowdin_api/api_resources/source_strings/resource.py

@@ -148,8 +148,8 @@ def edit_string(
         self,
         stringId: int,
         data: Iterable[SourceStringsPatchRequest],
-        updateOption: Optional[StringUpdateOption] = None,
         projectId: Optional[int] = None,
+        updateOption: Optional[StringUpdateOption] = None,
     ):
         """
         Edit String.
@@ -167,7 +167,7 @@ def edit_string(
         request_kwargs = {
             "method": "patch",
             "path": self.get_source_strings_path(projectId=projectId, stringId=stringId),
-            "request_data": data
+            "request_data": data,
         }
 
         if params:
@@ -178,8 +178,8 @@ def edit_string(
     def string_batch_operation(
         self,
         data: Iterable[StringBatchOperationPatchRequest],
-        updateOption: Optional[StringUpdateOption] = None,
         projectId: Optional[int] = None,
+        updateOption: Optional[StringUpdateOption] = None,
     ):
         """
         String Batch Operations.

crowdin_api/api_resources/source_strings/tests/test_source_strings_resources.py

@@ -208,17 +208,25 @@ def test_edit_string_with_update_option(self, m_request, base_absolut_url):
                 "value": "test",
                 "op": PatchOperation.REPLACE,
                 "path": SourceStringsPatchPath.TEXT,
-            }
+            },
         ]
 
         resource = self.get_resource(base_absolut_url)
-        assert resource.edit_string(projectId=1, stringId=2, data=data, updateOption=StringUpdateOption.KEEP_TRANSLATIONS) == "response"
+        assert (
+            resource.edit_string(
+                projectId=1,
+                stringId=2,
+                data=data,
+                updateOption=StringUpdateOption.KEEP_TRANSLATIONS,
+            )
+            == "response"
+        )
 
         m_request.assert_called_once_with(
             method="patch",
             request_data=data,
             params={
-                "updateOption": "keep_translations"
+                "updateOption": "keep_translations",
             },
             path=resource.get_source_strings_path(
                 projectId=1,
@@ -259,16 +267,24 @@ def test_string_batch_operation_with_update_option(self, m_request, base_absolut
                 "op": StringBatchOperations.REPLACE,
                 "path": StringBatchOperationsPath.IS_HIDDEN,
                 "value": True,
-            }
+            },
         ]
 
         resource = self.get_resource(base_absolut_url)
-        assert resource.string_batch_operation(projectId=1, data=data, updateOption=StringUpdateOption.KEEP_TRANSLATIONS) == "response"
+        assert (
+            resource.string_batch_operation(
+                projectId=1,
+                data=data,
+                updateOption=StringUpdateOption.KEEP_TRANSLATIONS,
+            )
+            == "response"
+        )
+
         m_request.assert_called_once_with(
             method="patch",
             path=resource.get_source_strings_path(1),
             request_data=data,
             params={
-                "updateOption": "keep_translations"
+                "updateOption": "keep_translations",
             },
         )