From 5345098219215c063a91f5bda456065e6fb56f93 Mon Sep 17 00:00:00 2001
From: James Kermode <james.kermode@gmail.com>
Date: Thu, 5 Aug 2021 18:50:47 +0100
Subject: [PATCH] Update python-package.yml

---
 .github/workflows/python-package.yml | 36 ++++++++++++++--------------
 1 file changed, 18 insertions(+), 18 deletions(-)

diff --git a/.github/workflows/python-package.yml b/.github/workflows/python-package.yml
index 7b20f6b..ee93d96 100644
--- a/.github/workflows/python-package.yml
+++ b/.github/workflows/python-package.yml
@@ -26,6 +26,17 @@ jobs:
       uses: actions/setup-python@v2
       with:
         python-version: ${{ matrix.python-version }}
+    - name: Install Python dependencies
+      run: |
+        python -m pip install --upgrade pip
+        python -m pip install flake8 pytest
+        python -m pip install -e . --verbose
+    - name: Lint with flake8
+      run: |
+        # stop the build if there are Python syntax errors or undefined names
+        flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics --exclude .git,QUIP
+        # exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
+        flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics --exclude=.git,QUIP
     - name: Build QUIP/libAtoms
       env: 
         QUIP_ARCH: linux_x86_64_gfortran
@@ -36,26 +47,10 @@ jobs:
         sudo apt-get install -y  gfortran libblas-dev liblapack-dev \
           openmpi-bin libopenmpi-dev netcdf-bin libnetcdf-dev libhdf5-serial-dev \
           python-numpy
-	git clone --recursive https://github.com/libAtoms/QUIP QUIP
+        git clone --recursive https://github.com/libAtoms/QUIP QUIP
         mkdir -p QUIP/build/${QUIP_ARCH}
         cp QUIP/.github/workflows/Makefile.inc QUIP/build/${QUIP_ARCH}/Makefile.inc
-        (cd QUIP && make libAtoms)
-    # # Uncomment to get SSH access for testing
-    # - name: Setup tmate session
-    #   if: failure()
-    #   uses: mxschmitt/action-tmate@v3
-    #   timeout-minutes: 15
-    - name: Install Python dependencies
-      run: |
-        python -m pip install --upgrade pip
-        python -m pip install flake8 pytest
-        python -m pip install -e . --verbose
-    - name: Lint with flake8
-      run: |
-        # stop the build if there are Python syntax errors or undefined names
-        flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics --exclude .git,QUIP
-        # exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
-        flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics --exclude=.git,QUIP
+        (cd QUIP && make libAtoms)	
     - name: Build Fortran executable
       env:
         QUIP_ARCH: linux_x86_64_gfortran
@@ -64,3 +59,8 @@ jobs:
     - name: Test with pytest
       run: |
         USE_FORTRAN=T pytest -v --ignore QUIP
+    # # Uncomment to get SSH access for testing
+    # - name: Setup tmate session
+    #   if: failure()
+    #   uses: mxschmitt/action-tmate@v3
+    #   timeout-minutes: 15