Enable code linting (#9)

* Install flake8 and run it in ci pipeline * conditionally run linting
Kazhuu · Jan 2, 2022 · 8ba1e70 · 8ba1e70
1 parent 4493e1c
commit 8ba1e70
Show file tree

Hide file tree

Showing 3 changed files with 40 additions and 3 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -5,6 +5,7 @@
 name: CI
 on:
   push:
+    branches: main
     paths-ignore:
     - 'LICENSE'
     - 'README.md'
@@ -29,6 +30,10 @@ jobs:
         sudo add-apt-repository ppa:deadsnakes/ppa
         sudo apt-get update
         sudo apt-get install ${PYTHON}
-        ${PYTHON} -m pip install graphviz
+        ${PYTHON} -m pip install pipenv
+        pipenv install --dev
     - name: Run tests
-      run: ${PYTHON} -m unittest
+      run: pipenv run python -m unittest
+    - name: Linting
+      if: ${{ matrix.os == 'ubuntu-latest' }}
+      run: pipenv run flake8
diff --git a/Pipfile b/Pipfile
@@ -8,3 +8,4 @@ graphviz = "*"
 
 [dev-packages]
 ipdb = "*"
+flake8 = "*"
diff --git a/Pipfile.lock b/Pipfile.lock
Original file line number	Diff line number	Diff line change
Expand Up		@@ -8,3 +8,4 @@ graphviz = "*"

		[dev-packages]
		ipdb = "*"
		flake8 = "*"