Merge pull request #1 from jpetrucciani/actions

adding github actions, fixing prospector issues

.github/workflows/test.yml

@@ -0,0 +1,29 @@
+name: test
+
+on: [push, pull_request]
+
+jobs:
+  archives:
+    runs-on: ubuntu-18.04
+    steps:
+      - uses: actions/checkout@v1
+      - uses: jpetrucciani/archives-check@master
+        with:
+          path: 'archives/'
+  prospector:
+    runs-on: ubuntu-18.04
+    steps:
+      - uses: actions/checkout@v1
+      - uses: jpetrucciani/prospector-check@master
+  mypy:
+    runs-on: ubuntu-18.04
+    steps:
+      - uses: actions/checkout@v1
+      - uses: jpetrucciani/mypy-check@master
+  black:
+    runs-on: ubuntu-18.04
+    steps:
+      - uses: actions/checkout@v1
+      - uses: jpetrucciani/black-check@master
+        with:
+          path: 'archives/'

.prospector.yaml

@@ -7,6 +7,7 @@ ignore-patterns:
   - (^|/)\..+
   - .*\.html
   - docs/.*
+  - extra/.*
   - tmp\.py
 
 pylint:
@@ -22,9 +23,9 @@ pylint:
     - wrong-import-order
 
   options:
-    max-args: 10
+    max-args: 20
     max-locals: 100
-    max-returns: 6
+    max-returns: 10
     max-branches: 50
     max-statements: 180
     max-parents: 10
@@ -67,11 +68,13 @@ pep257:
     - D105
     - D107
     - D200
+    - D202
     - D203
     - D205
     - D212
     - D204
     - D300
+    - D301
     - D400
     - D401
     - D404

archives/archives.py

@@ -343,7 +343,7 @@ def archives(
     verbose: bool,
     include: str,
     exclude: str,
-    format: str,
+    format: str,  # pylint: disable=redefined-builtin
     disable: str,
     list_rules: bool,
     list_tags: bool,

archives/models/python.py

@@ -35,8 +35,7 @@ def parse_elt(elt: Union[ast3.Name, ast3.Subscript]) -> str:
             return "{}[{}]".format(
                 name, ", ".join(parse_elt(x) for x in value.elts)  # type: ignore
             )
-        else:
-            debug(value)
+        debug(value)
     return ""
 
 
@@ -96,7 +95,7 @@ def __init__(self, doc_string: ast3.Expr, doc_type: Type) -> None:
         self.value = doc_string.value.s.strip()  # type: ignore
         desc = Tags.DESC.regex.search(self.value)
         ret = Tags.RETURN.regex.search(self.value)
-        cc = Tags.CC.regex.search(self.value)
+        cc = Tags.CC.regex.search(self.value)  # pylint: disable=invalid-name
         author = Tags.AUTHOR.regex.search(self.value)
         todo = Tags.TODO.regex.search(self.value)
 
@@ -113,7 +112,7 @@ def __init__(self, doc_string: ast3.Expr, doc_type: Type) -> None:
         self.links = {x: y for x, y in Tags.LINK.regex.findall(self.value)}
         self.ret = ret[1] if ret else ""
         self.author = author[1] if author else ""
-        self.cc = int(cc[1] if cc else -1)
+        self.cc = int(cc[1] if cc else -1)  # pylint: disable=invalid-name
 
         self.notes = [x for x in Tags.NOTE.regex.findall(self.value)]
         self.warnings = [x for x in Tags.WARN.regex.findall(self.value)]

archives/utils/files.py

@@ -68,7 +68,7 @@ def find_project_root(sources: Iterable[str]) -> Path:
         if (directory / ".git").is_dir():
             return directory
 
-    return directory
+    return directory  # pylint: disable=undefined-loop-variable
 
 
 def path_empty(src: Tuple[str], ctx: click.Context) -> None:
@@ -97,7 +97,7 @@ def decode_bytes(src: bytes) -> Tuple[str, str, str]:
     if not lines:
         return "", encoding, "\n"
 
-    newline = "\r\n" if b"\r\n" == lines[0][-2:] else "\n"
+    newline = "\r\n" if lines[0][-2:] == b"\r\n" else "\n"
     srcbuf.seek(0)
     with io.TextIOWrapper(srcbuf, encoding) as tiow:
         return tiow.read(), encoding, newline