Merge pull request #182 from SchrodingersGat/github-workflow

Adds github workflow for PEP style checks

.github/release.yml

@@ -0,0 +1,32 @@
+# .github/release.yml
+
+changelog:
+  exclude:
+    labels:
+      - translation
+      - documentation
+  categories:
+    - title: Breaking Changes
+      labels:
+        - Semver-Major
+        - breaking
+    - title: Security Patches
+      labels:
+        - security
+    - title: New Features
+      labels:
+        - Semver-Minor
+        - enhancement
+    - title: Bug Fixes
+      labels:
+        - Semver-Patch
+        - bug
+    - title: Devops / Setup Changes
+      labels:
+        - docker
+        - setup
+        - demo
+        - CI
+    - title: Other Changes
+      labels:
+        - "*"

.github/workflows/build.yaml

@@ -0,0 +1,27 @@
+name: Build Package
+
+on: ["push", "pull_request"]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+
+    strategy:
+      max-parallel: 4
+      matrix:
+        python-version: [3.8]
+
+    steps:
+      - name: Checkout Code
+        uses: actions/checkout@v2
+      - name: Set up Python ${{ matrix.python-version }}
+        uses: actions/setup-python@v2
+        with:
+          python-version: ${{ matrix.python-version }}
+      - name: Install Deps
+        run: |
+          pip install -U -r test/requirements.txt
+      - name: Build Python Package
+        run: |
+          pip install --upgrade pip wheel setuptools
+          python setup.py bdist_wheel --universal

.github/workflows/pep.yaml

@@ -0,0 +1,33 @@
+name: PEP Style Checks
+
+on: ["push", "pull_request"]
+
+jobs:
+  style:
+    runs-on: ubuntu-latest
+
+    env:
+      GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+
+    strategy:
+      max-parallel: 4
+      matrix:
+        python-version: [3.8]
+
+    steps:
+      - name: Checkout Code
+        uses: actions/checkout@v2
+      - name: Set up Python ${{ matrix.python-version }}
+        uses: actions/setup-python@v2
+        with:
+          python-version: ${{ matrix.python-version }}
+      - name: Install Deps
+        run: |
+          pip install -U -r test/requirements.txt
+      - name: Style Checks
+        run: |
+          flake8 .
+          bash ./run-tests.sh
+      - name: Upload Report
+        run: |
+          coveralls --service=github

kibom/html_writer.py

@@ -178,7 +178,7 @@ def WriteHTML(filename, groups, net, headings, head_names, prefs):
 
             for i, group in enumerate(groups):
 
-                if not(prefs.ignoreDNF and not group.isFitted()):
+                if not (prefs.ignoreDNF and not group.isFitted()):
                     continue
 
                 row = group.getRow(headings)

setup.cfg

@@ -6,5 +6,5 @@ ignore =
 	E501, E722,
 	# - C901 - function is too complex
 	C901,
-exclude = .git,__pycache__,*/migrations/*,./build,
+exclude = .git,__pycache__,*/migrations/*,./build, ./test,
 max-complexity = 20

test/requirements.txt

@@ -1,5 +1,6 @@
 # Python packages required for unit testing
 
-flake8==3.3.0                   # PEP checking
-coverage==4.0.3                 # Unit test coverage
-python-coveralls==2.9.1         # Coveralls linking (for Travis)
+flake8>=6.0.0                   # PEP checking
+coverage                        # Unit test coverage
+coveralls>=3.3.0                # Coveralls.io linking
+xlsxwriter>=3.0.0               # Excel file writing