Merge PR #210 from knocte/wip/putCommitLintLast

* Move commit-lint at the end cecause it's less important than
the others.
* Move prettier after fantomas & FSharpLint because it's less
important than them.
* Move commit check1by1 to later because it's the one more
likely to break when WIP working on a PR.

Author: knocte

.github/workflows/CI.yml

@@ -172,11 +172,11 @@ jobs:
         run: |
           apt update && apt install --yes sudo
 
-          sudo apt install --yes --no-install-recommends npm curl
-
+          # otherwise it gives error "Error: Input 'submodules' not supported when falling back to download using the GitHub REST API. To create a local Git repository instead, add Git 2.18 or higher to the PATH."
           sudo apt install --yes --no-install-recommends git
-          # workaround for https://github.com/actions/runner/issues/2033
-          git config --global --add safe.directory '*'
+
+          # otherwise it gives error "fatal: unable to access 'https://github.com/nblockchain/conventions/': server certificate verification failed. CAfile: none CRLfile: none"
+          sudo apt install --yes --no-install-recommends ca-certificates
       - uses: nblockchain/conventions@master
         with:
           uses: actions/checkout@v${{ env.CHECKOUT_ACTION_VERSION }}
@@ -185,21 +185,6 @@ jobs:
               "submodules": "recursive",
               "fetch-depth": 0
             }
-      - name: Print versions
-        run: |
-          git --version
-          node --version
-          npm --version
-      - name: Validate current commit (last commit) with commitlint
-        if: github.event_name == 'push'
-        run: ./commitlint.sh --last --verbose
-      - name: Validate PR commits with commitlint
-        if: github.event_name == 'pull_request'
-        run: |
-          ./commitlint.sh --verbose \
-            --from ${{ github.event.pull_request.head.sha }}~${{ github.event.pull_request.commits }} \
-            --to ${{ github.event.pull_request.head.sha }}
-
       - name: Setup .NET
         run: |
           # We need to install `ca-certificates`, otherwise we get these errors in the CI:
@@ -226,9 +211,6 @@ jobs:
         run: dotnet fsi scripts/unpinnedNugetPackageReferenceVersionsInFSharpScripts.fsx
       - name: Check there are no unpinned versions in `dotnet tool install` commands
         run: dotnet fsi scripts/unpinnedDotnetToolInstallVersions.fsx
-      - name: Check commits 1 by 1
-        if: github.event_name == 'pull_request'
-        run: dotnet fsi scripts/checkCommits1by1.fsx
       - name: Check there are no inconsistent versions GitHubCI files
         run: dotnet fsi scripts/inconsistentVersionsInGitHubCI.fsx
       - name: Check there are no inconsistent versions in nuget package references of F# scripts
@@ -241,12 +223,39 @@ jobs:
           dotnet fsi scripts/inconsistentNugetVersionsInDotNetProjects.fsx
       - name: Check there are no inconsistent versions in nuget package references of F# scripts and projects
         run: dotnet fsi scripts/inconsistentNugetVersionsInDotNetProjectsAndFSharpScripts.fsx
+      - name: FSharpLint
+        run: |
+          dotnet fsi scripts/runFSharpLint.fsx
+      - name: Run git+githubCI workaround
+        run: |
+          # workaround for https://github.com/actions/runner/issues/2033
+          git config --global --add safe.directory '*'
+      - name: fantomless
+        run: |
+          dotnet tool install fantomless-tool --version 4.7.997-prerelease
+          dotnet fantomless --recurse .
+          git diff --exit-code
+      - name: Check commits 1 by 1
+        if: github.event_name == 'pull_request'
+        run: dotnet fsi scripts/checkCommits1by1.fsx
+
+      - name: Install commitlint&prettier's required dependencies
+        run: |
+          sudo apt install --yes --no-install-recommends npm
+      - name: Print versions
+        run: |
+          git --version
+          node --version
+          npm --version
+
+      - name: Run git+githubCI workaround
+        run: |
+          # We need this step so we can change the files using `npx prettier --write` in the next step.
+          # Otherwise we get permission denied error in the CI.
+          sudo chmod 777 --recursive .
       - name: Install prettier
         run: npm install --verbose [email protected]
-      - name: Change file permissions
-        # We need this step so we can change the files using `npx prettier --write` in the next step.
-        # Otherwise we get permission denied error in the CI.
-        run: sudo chmod 777 --recursive .
+
       - name: Run "prettier" to check the style of our TypeScript and YML code
         run: |
           sudo npx prettier --quote-props=consistent --write './**/*.ts'
@@ -258,11 +267,13 @@ jobs:
           # run the following command to ignore package.json file
           git restore package.json
           git diff --exit-code
-      - name: FSharpLint
-        run: |
-          dotnet fsi scripts/runFSharpLint.fsx
-      - name: fantomless
+
+      - name: Validate current commit (last commit) with commitlint
+        if: github.event_name == 'push'
+        run: ./commitlint.sh --last --verbose
+      - name: Validate PR commits with commitlint
+        if: github.event_name == 'pull_request'
         run: |
-          dotnet tool install fantomless-tool --version 4.7.997-prerelease
-          dotnet fantomless --recurse .
-          git diff --exit-code
+          ./commitlint.sh --verbose \
+            --from ${{ github.event.pull_request.head.sha }}~${{ github.event.pull_request.commits }} \
+            --to ${{ github.event.pull_request.head.sha }}