style: flake8 cleanup

Makefile

@@ -22,7 +22,7 @@ clean-test: ## remove test and coverage artifacts
 	rm -fr .pytest_cache
 
 lint: ## check style with flake8
-	flake8 micropy tests --exclude micropy/lib
+	flake8 micropy tests --config=setup.cfg
 
 test: ## run tests quickly with the default Python
 	py.test

micropy/cli.py

@@ -38,7 +38,8 @@ def reload(project_name=''):
 
 
 @stubs.command()
-@click.argument('path', required=True, type=click.Path(exists=True, file_okay=False, resolve_path=True))
+@click.argument('path', required=True, type=click.Path(
+    exists=True, file_okay=False, resolve_path=True))
 def add(path):
     """Add stubs"""
     return mp.add_stub(path)

micropy/exceptions.py

@@ -2,8 +2,6 @@
 
 """Micropy Exceptions"""
 
-from micropy.logger import Log
-
 
 class StubError(Exception):
     """Exception for any errors raised by stubs"""
@@ -20,7 +18,8 @@ class StubValidationError(StubError):
 
     def __init__(self, stub, errors):
         errs = '\n'.join(errors)
-        msg = f"Stub at [{stub.path}] encountered the following validation errors: {errs}"
+        msg = f"Stub at [{stub.path}] encountered \
+            the following validation errors: {errs}"
         super().__init__(stub, message=msg)
 
     def __str__(self):

micropy/logger.py

@@ -44,7 +44,8 @@ class ServiceLog:
     """
     LOG_FILE = Path.home() / '.micropy' / 'micropy.log'
 
-    def __init__(self, service_name='MicroPy', base_color='bright_green', **kwargs):
+    def __init__(
+            self, service_name='MicroPy', base_color='bright_green', **kwargs):
         self.parent = kwargs.get('parent', None)
         self.LOG_FILE.parent.mkdir(exist_ok=True)
         logging.basicConfig(level=logging.DEBUG,
@@ -160,7 +161,9 @@ def exception(self, error, **kwargs):
 
         """
         logging.exception(str(error))
-        return self.echo(str(error), title_color='red', title_bold=True, **kwargs)
+        return self.echo(
+            str(error),
+            title_color='red', title_bold=True, **kwargs)
 
     def success(self, msg, **kwargs):
         """Prints message with success formatting
@@ -201,7 +204,8 @@ def prompt(self, msg, **kwargs):
         nl_default = kwargs.get('default', None)
         msg = self.parse_msg(msg)
         msg = msg + \
-            style(f"\n Press Enter to Use: [{nl_default}]", dim=True) if nl_default and len(
+            style(f"\n Press Enter to Use: [{nl_default}]", dim=True) if \
+            nl_default and len(
                 nl_default) > 0 else msg
         title = self.get_service()
         suffix = style('\u27a4 ', fg=self.accent_color)
@@ -222,7 +226,8 @@ def confirm(self, msg, **kwargs):
         title = self.get_service()
         suffix = style('\u27a4 ', fg=self.accent_color)
         secho(f"{title} ", nl=False)
-        return confirm(msg, show_default="[y/N] ", prompt_suffix=suffix, **kwargs)
+        return confirm(
+            msg, show_default="[y/N] ", prompt_suffix=suffix, **kwargs)
 
     def clear(self):
         """Clears terminal screen"""

micropy/main.py

@@ -4,7 +4,6 @@
 
 import tempfile
 from pathlib import Path
-from shutil import copytree
 
 from rshell import main as rsh
 
@@ -59,7 +58,6 @@ def add_stub(self, path):
             stub = Stub(path, copy_to=stub_out)
         except StubValidationError:
             self.log.error(f"{stub_path.name} is not a valid stub!")
-            pass
         else:
             self.STUBS.append(stub)
             self.log.debug(f"Added New Stub: {stub}")
@@ -93,10 +91,13 @@ def create_stubs(self, port):
         self.log.success("Done!")
         self.log.info("Downloading Stubs...")
         stub_name = rsh.auto(
-            rsh.listdir_stat, f'{dev.name_path}/stubs', show_hidden=False)[0][0]
+            rsh.listdir_stat, f'{dev.name_path}/stubs',
+            show_hidden=False)[0][0]
         with tempfile.TemporaryDirectory() as tmpdir:
-            rsh.rsync(f"{dev.name_path}/stubs", tmpdir, recursed=True, mirror=False,
-                      dry_run=False, print_func=lambda *args: None, sync_hidden=False)
+            rsh.rsync(
+                f"{dev.name_path}/stubs", tmpdir, recursed=True, mirror=False,
+                dry_run=False, print_func=lambda * args: None,
+                sync_hidden=False)
             stub_path = Path(tmpdir) / stub_name
             self.add_stub(stub_path)
         self.log.success("Done!")

micropy/project.py

@@ -70,9 +70,11 @@ def load_stubs(self):
         vs_stubs = [f'"{s.path}"' for s in stubs]
         self.log.info(f"Found $[{len(stubs)}] stubs, injecting...")
         lint_stubs = prompt.checkbox(
-            "Which stubs would you like pylint to load?", choices=[str(i) for i in stubs]).ask()
+            "Which stubs would you like pylint to load?",
+            choices=[str(i) for i in stubs]).ask()
         pylint_stubs = [
-            f'sys.path.insert(1,"{str(stub.path)}")' for stub in stubs if str(stub) in lint_stubs]
+            f'sys.path.insert(1,"{str(stub.path)}")' for stub in stubs
+            if str(stub) in lint_stubs]
         vscode_sub = {
             'stubs': ',\n'.join(vs_stubs)
         }

micropy/stubs/stubs.py

@@ -3,11 +3,11 @@
 import json
 from pathlib import Path
 
-from jsonschema import Draft7Validator, ValidationError
+from jsonschema import Draft7Validator
 from shutil import copytree
 
 from micropy.logger import Log
-from micropy.exceptions import StubError, StubValidationError
+from micropy.exceptions import StubValidationError
 
 
 class Stub:
@@ -70,7 +70,9 @@ def copy_to(self, dest):
         return self
 
     def __repr__(self):
-        return f"Stub(machine={self.machine}, nodename={self.nodename}, release={self.release}, sysname={self.sysname}, version={self.version}, modules={self.modules})"
+        return f"Stub(machine={self.machine}, nodename={self.nodename}, \
+                release={self.release}, sysname={self.sysname}, \
+                version={self.version}, modules={self.modules})"
 
     def __str__(self):
         return f"{self.sysname}@{self.version}"

setup.cfg

@@ -15,7 +15,7 @@ replace = __version__ = '{new_version}'
 universal = 1
 
 [flake8]
-exclude = docs,micropy/lib
+exclude = docs,micropy/lib,tests/data
 
 [aliases]
 test = pytest

tests/conftest.py

@@ -3,7 +3,6 @@
 
 import pytest
 import micropy
-from pathlib import Path
 import questionary
 
 

tests/test_main.py

@@ -1,10 +1,6 @@
 # -*- coding: utf-8 -*-
 
-import pytest
 from micropy import main
-from pathlib import Path
-import os
-import requests
 
 
 def test_setup(mock_micropy_path):

tests/test_stubs.py

@@ -1,9 +1,7 @@
 # -*- coding: utf-8 -*-
 
 import pytest
-from pathlib import Path
 from micropy import stubs, exceptions
-import json
 
 
 def test_bad_stub(tmp_path):