Defined linter as a separate job (firebase#398)

hiranya911 · web-flow · commit ffebd3cd42aa · 2020-01-30T13:34:30.000-08:00
* Defined linter as a separate job

* Added missing env config

* Fixed pylint install directive

* Installing all dependencies

* Disabling fail fast for build matrix

* Merged with master
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -4,9 +4,9 @@ on: [push, pull_request]
 
 jobs:
   build:
-
     runs-on: ubuntu-latest
     strategy:
+      fail-fast: false
       matrix:
         python: [3.5, 3.6, 3.7, pypy3]
 
@@ -20,11 +20,7 @@ jobs:
       run: |
         python -m pip install --upgrade pip
         pip install -r requirements.txt
-    - name: Lint with pylint
-      if: matrix.python == '3.7'
-      run: ./lint.sh all
     - name: Test with pytest
-      if: success() || failure()
       run: pytest
     - name: Set up Node.js 10
       uses: actions/setup-node@v1
@@ -34,3 +30,18 @@ jobs:
       run: |
         npm install -g firebase-tools
         firebase emulators:exec --only database --project fake-project-id 'pytest integration/test_db.py'
+
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v1
+    - name: Set up Python 3.7
+      uses: actions/setup-python@v1
+      with:
+        python-version: 3.7
+    - name: Install dependencies
+      run: |
+        python -m pip install --upgrade pip
+        pip install -r requirements.txt
+    - name: Lint with pylint
+      run: ./lint.sh all
