Merge pull request #12 from makermelissa/main

Updated so checks are passing

Author: makermelissa

.pre-commit-config.yaml

@@ -4,15 +4,15 @@
 
 repos:
 -   repo: https://github.com/python/black
-    rev: 19.10b0
+    rev: 21.6b0
     hooks:
     - id: black
 -   repo: https://github.com/fsfe/reuse-tool
-    rev: latest
+    rev: v0.12.1
     hooks:
     - id: reuse
 -   repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v2.3.0
+    rev: v4.0.1
     hooks:
     -   id: check-yaml
     -   id: end-of-file-fixer

adafruit_shell.py

@@ -55,7 +55,11 @@ def select_n(message, selections):
 
         for index, selection in enumerate(selections):
             options.append(
-                {"selector": str(index + 1), "prompt": selection, "return": index + 1,}
+                {
+                    "selector": str(index + 1),
+                    "prompt": selection,
+                    "return": index + 1,
+                }
             )
         return prompt.options(message, options)
 
@@ -66,9 +70,11 @@ def run_command(self, cmd, suppress_message=False, return_output=False):
         original_stdout = sys.stdout
         original_stderr = sys.stderr
         try:
+            # pylint: disable=consider-using-with
             proc = subprocess.Popen(
                 cmd, shell=True, stdout=subprocess.PIPE, stderr=subprocess.PIPE
             )
+            # pylint: enable=consider-using-with
             full_output = ""
             while True:
                 output = proc.stdout.readline()
@@ -391,9 +397,8 @@ def write_text_file(self, path, content, append=True):
             content = "\n" + content
         else:
             mode = "w"
-        service_file = open(self.path(path), mode)
-        service_file.write(content)
-        service_file.close()
+        with open(self.path(path), mode) as service_file:
+            service_file.write(content)
 
     @staticmethod
     def is_python3():

setup.py

@@ -34,7 +34,10 @@
     # Author details
     author="Adafruit Industries",
     author_email="[email protected]",
-    install_requires=["clint", "Adafruit-PlatformDetect",],
+    install_requires=[
+        "clint",
+        "Adafruit-PlatformDetect",
+    ],
     # Choose your license
     license="MIT",
     # See https://pypi.python.org/pypi?%3Aaction=list_classifiers