attempt to format all files with ruff? idk

eleanorjboyd · Oct 31, 2024 · ebe56be · ebe56be
1 parent 1bc72ba
commit ebe56be
Show file tree

Hide file tree

Showing 22 changed files with 104 additions and 94 deletions.
diff --git a/.github/actions/lint/action.yml b/.github/actions/lint/action.yml
@@ -46,5 +46,4 @@ runs:
         python -m pip install -U ruff
         python -m ruff check .
         python -m ruff format --check
-      working-directory: python_files
       shell: bash
diff --git a/build/ci/addEnvPath.py b/build/ci/addEnvPath.py
@@ -1,26 +1,27 @@
 # Copyright (c) Microsoft Corporation. All rights reserved.
 # Licensed under the MIT License.
 
-#Adds the virtual environment's executable path to json file
+# Adds the virtual environment's executable path to json file
 
 import json
 import sys
 import os.path
+
 jsonPath = sys.argv[1]
 key = sys.argv[2]
 
 if os.path.isfile(jsonPath):
-    with open(jsonPath, 'r') as read_file:
+    with open(jsonPath, "r") as read_file:
         data = json.load(read_file)
 else:
     directory = os.path.dirname(jsonPath)
     if not os.path.exists(directory):
         os.makedirs(directory)
-        with open(jsonPath, 'w+') as read_file:
+        with open(jsonPath, "w+") as read_file:
             data = {}
     data = {}
-with open(jsonPath, 'w') as outfile:
-    if key == 'condaExecPath':
+with open(jsonPath, "w") as outfile:
+    if key == "condaExecPath":
         data[key] = sys.argv[3]
     else:
         data[key] = sys.executable

diff --git a/src/test/multiRootWkspc/disableLinters/file.py b/src/test/multiRootWkspc/disableLinters/file.py
@@ -2,6 +2,7 @@
 
 __revision__ = None
 
+
 class Foo(object):
     """block-disable test"""
 
@@ -15,73 +16,71 @@ def meth1(self, arg):
     def meth2(self, arg):
         """and this one not"""
         # pylint: disable=unused-argument
-        print (self\
-              + "foo")
+        print(self + "foo")
 
     def meth3(self):
         """test one line disabling"""
         # no error
-        print (self.bla) # pylint: disable=no-member
+        print(self.bla)  # pylint: disable=no-member
         # error
-        print (self.blop)
+        print(self.blop)
 
     def meth4(self):
         """test re-enabling"""
         # pylint: disable=no-member
         # no error
-        print (self.bla)
-        print (self.blop)
+        print(self.bla)
+        print(self.blop)
         # pylint: enable=no-member
         # error
-        print (self.blip)
+        print(self.blip)
 
     def meth5(self):
         """test IF sub-block re-enabling"""
         # pylint: disable=no-member
         # no error
-        print (self.bla)
+        print(self.bla)
         if self.blop:
             # pylint: enable=no-member
             # error
-            print (self.blip)
+            print(self.blip)
         else:
             # no error
-            print (self.blip)
+            print(self.blip)
         # no error
-        print (self.blip)
+        print(self.blip)
 
     def meth6(self):
         """test TRY/EXCEPT sub-block re-enabling"""
         # pylint: disable=no-member
         # no error
-        print (self.bla)
+        print(self.bla)
         try:
             # pylint: enable=no-member
             # error
-            print (self.blip)
-        except UndefinedName: # type: ignore # pylint: disable=undefined-variable  # noqa: F821
+            print(self.blip)
+        except UndefinedName:  # type: ignore # pylint: disable=undefined-variable  # noqa: F821
             # no error
-            print (self.blip)
+            print(self.blip)
         # no error
-        print (self.blip)
+        print(self.blip)
 
     def meth7(self):
         """test one line block opening disabling"""
-        if self.blop: # pylint: disable=no-member
+        if self.blop:  # pylint: disable=no-member
             # error
-            print (self.blip)
+            print(self.blip)
         else:
             # error
-            print (self.blip)
+            print(self.blip)
         # error
-        print (self.blip)
-
+        print(self.blip)
 
     def meth8(self):
         """test late disabling"""
         # error
-        print (self.blip)
+        print(self.blip)
         # pylint: disable=no-member
         # no error
-        print (self.bla)
-        print (self.blop)
+        print(self.bla)
+        print(self.blop)
diff --git a/src/test/multiRootWkspc/parent/child/childFile.py b/src/test/multiRootWkspc/parent/child/childFile.py
@@ -2,6 +2,7 @@
 
 __revision__ = None
 
+
 class Child2Class(object):
     """block-disable test"""
 
@@ -10,4 +11,4 @@ def __init__(self):
 
     def meth1OfChild(self, arg):
         """this issues a message"""
-        print (self)
+        print(self)
diff --git a/src/test/multiRootWkspc/workspace2/workspace2File.py b/src/test/multiRootWkspc/workspace2/workspace2File.py
@@ -2,6 +2,7 @@
 
 __revision__ = None
 
+
 class Workspace2Class(object):
     """block-disable test"""
 
@@ -10,4 +11,4 @@ def __init__(self):
 
     def meth1OfWorkspace2(self, arg):
         """this issues a message"""
-        print (self)
+        print(self)
diff --git a/src/test/multiRootWkspc/workspace3/file.py b/src/test/multiRootWkspc/workspace3/file.py
@@ -2,6 +2,7 @@
 
 __revision__ = None
 
+
 class Foo(object):
     """block-disable test"""
 
@@ -15,73 +16,71 @@ def meth1(self, arg):
     def meth2(self, arg):
         """and this one not"""
         # pylint: disable=unused-argument
-        print (self\
-              + "foo")
+        print(self + "foo")
 
     def meth3(self):
         """test one line disabling"""
         # no error
-        print (self.bla) # pylint: disable=no-member
+        print(self.bla)  # pylint: disable=no-member
         # error
-        print (self.blop)
+        print(self.blop)
 
     def meth4(self):
         """test re-enabling"""
         # pylint: disable=no-member
         # no error
-        print (self.bla)
-        print (self.blop)
+        print(self.bla)
+        print(self.blop)
         # pylint: enable=no-member
         # error
-        print (self.blip)
+        print(self.blip)
 
     def meth5(self):
         """test IF sub-block re-enabling"""
         # pylint: disable=no-member
         # no error
-        print (self.bla)
+        print(self.bla)
         if self.blop:
             # pylint: enable=no-member
             # error
-            print (self.blip)
+            print(self.blip)
         else:
             # no error
-            print (self.blip)
+            print(self.blip)
         # no error
-        print (self.blip)
+        print(self.blip)
 
     def meth6(self):
         """test TRY/EXCEPT sub-block re-enabling"""
         # pylint: disable=no-member
         # no error
-        print (self.bla)
+        print(self.bla)
         try:
             # pylint: enable=no-member
             # error
-            print (self.blip)
-        except UndefinedName: # type: ignore # pylint: disable=undefined-variable  # noqa: F821
+            print(self.blip)
+        except UndefinedName:  # type: ignore # pylint: disable=undefined-variable  # noqa: F821
             # no error
-            print (self.blip)
+            print(self.blip)
         # no error
-        print (self.blip)
+        print(self.blip)
 
     def meth7(self):
         """test one line block opening disabling"""
-        if self.blop: # pylint: disable=no-member
+        if self.blop:  # pylint: disable=no-member
             # error
-            print (self.blip)
+            print(self.blip)
         else:
             # error
-            print (self.blip)
+            print(self.blip)
         # error
-        print (self.blip)
-
+        print(self.blip)
 
     def meth8(self):
         """test late disabling"""
         # error
-        print (self.blip)
+        print(self.blip)
         # pylint: disable=no-member
         # no error
-        print (self.bla)
-        print (self.blop)
+        print(self.bla)
+        print(self.blop)
diff --git a/src/test/pythonFiles/dummy.py b/src/test/pythonFiles/dummy.py
@@ -1 +1 @@
-#dummy file to be opened by Test VS Code instance, so that Python Configuration (workspace configuration will be initialized)
+# dummy file to be opened by Test VS Code instance, so that Python Configuration (workspace configuration will be initialized)
diff --git a/src/test/python_files/datascience/simple_note_book.py b/src/test/python_files/datascience/simple_note_book.py
@@ -1,7 +1,7 @@
 # %%
 import os.path
-dir_path = os.path.dirname(os.path.realpath(__file__))
 
-with open(os.path.join(dir_path, 'ds.log'), 'a') as fp:
-    fp.write('Hello World')
+dir_path = os.path.dirname(os.path.realpath(__file__))
 
+with open(os.path.join(dir_path, "ds.log"), "a") as fp:
+    fp.write("Hello World")
diff --git a/src/test/python_files/dummy.py b/src/test/python_files/dummy.py
@@ -1 +1 @@
-#dummy file to be opened by Test VS Code instance, so that Python Configuration (workspace configuration will be initialized)
+# dummy file to be opened by Test VS Code instance, so that Python Configuration (workspace configuration will be initialized)
diff --git a/src/test/python_files/shebang/plain.py b/src/test/python_files/shebang/plain.py
@@ -1,2 +1 @@
-
 print("dummy")
diff --git a/src/testMultiRootWkspc/parent/child/childFile.py b/src/testMultiRootWkspc/parent/child/childFile.py
@@ -2,6 +2,7 @@
 
 __revision__ = None
 
+
 class Child2Class(object):
     """block-disable test"""
 
@@ -10,4 +11,4 @@ def __init__(self):
 
     def meth1OfChild(self, arg):
         """this issues a message"""
-        print (self)
+        print(self)
diff --git a/src/testMultiRootWkspc/smokeTests/create_delete_file.py b/src/testMultiRootWkspc/smokeTests/create_delete_file.py
@@ -1,5 +1,5 @@
-with open('smart_send_smoke.txt', 'w') as f:
-    f.write('This is for smart send smoke test')
+with open("smart_send_smoke.txt", "w") as f:
+    f.write("This is for smart send smoke test")
 import os
 
-os.remove('smart_send_smoke.txt')
+os.remove("smart_send_smoke.txt")
diff --git a/src/testMultiRootWkspc/smokeTests/definitions.py b/src/testMultiRootWkspc/smokeTests/definitions.py
@@ -1,23 +1,28 @@
 from contextlib import contextmanager
 
+
 def my_decorator(fn):
     """
     This is my decorator.
     """
+
     def wrapper(*args, **kwargs):
         """
         This is the wrapper.
         """
         return 42
+
     return wrapper
 
+
 @my_decorator
 def thing(arg):
     """
     Thing which is decorated.
     """
     pass
 
+
 @contextmanager
 def my_context_manager():
     """
@@ -27,5 +32,6 @@ def my_context_manager():
     yield
     print("after")
 
+
 with my_context_manager():
     thing(19)
diff --git a/src/testMultiRootWkspc/smokeTests/testExecInTerminal.py b/src/testMultiRootWkspc/smokeTests/testExecInTerminal.py
@@ -2,11 +2,11 @@
 import sys
 import os
 
-optlist, args = getopt.getopt(sys.argv, '')
+optlist, args = getopt.getopt(sys.argv, "")
 
 # If the caller has not specified the output file, create one for them with
 # the same name as the caller script, but with a .log extension.
-log_file = os.path.splitext(sys.argv[0])[0] + '.log'
+log_file = os.path.splitext(sys.argv[0])[0] + ".log"
 
 # If the output file is given, use that instead.
 if len(args) == 2: