Use shallow fetch depth where possible. Increases build speed.

Another yaml fix Further indentation fixes. Fix yaml spacing.
rasky · Jan 25, 2023 · 30bba98 · 30bba98
1 parent ef48726
commit 30bba98
Show file tree

Hide file tree

Showing 3 changed files with 32 additions and 28 deletions.
diff --git a/.github/workflows/build-documentation.yml b/.github/workflows/build-documentation.yml
@@ -18,6 +18,8 @@ jobs:
     steps:
       - name: Checkout
         uses: actions/checkout@v3
+        with:
+          fetch-depth: 1 # Using a shallow checkout. Change to `0` if a full fetch is required.
 
       - name: Set variables
         id: vars

diff --git a/.github/workflows/build-tool-windows.yml b/.github/workflows/build-tool-windows.yml
@@ -6,36 +6,36 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-       include: [
-         { sys: mingw64, arch: x86_64, build: tools},
-         { sys: mingw32, arch: i686,   build: tools}
-       ]
+        include: [
+          { sys: mingw64, arch: x86_64, build: tools},
+          { sys: mingw32, arch: i686,   build: tools}
+        ]
     steps:
-     - uses: msys2/setup-msys2@v2
-       with:
-         msystem: ${{matrix.sys}}
-         install: >-
-           mingw-w64-${{ matrix.arch }}-libpng
-           base-devel
-           mingw-w64-${{ matrix.arch }}-toolchain
-         update: true
+      - uses: msys2/setup-msys2@v2
+        with:
+          msystem: ${{matrix.sys}}
+          install: >-
+            mingw-w64-${{ matrix.arch }}-libpng
+            base-devel
+            mingw-w64-${{ matrix.arch }}-toolchain
+          update: true
 
-     - uses: actions/checkout@v3
-       with:
-         fetch-depth: 0
+      - uses: actions/checkout@v3
+        with:
+          fetch-depth: 1 # Using a shallow checkout. Change to `0` if a full fetch is required.
 
-     - name: Correct MSYS2 pthread.h to allow static libraries (otherwise you would need to use a lib DLL, rather than it being built into the EXE.)
-       shell: msys2 {0}
-       run: |
-         sed -z 's/#else\n#define WINPTHREAD_API __declspec(dllimport)/#else\n#define WINPTHREAD_API/' /${{matrix.sys}}/include/pthread.h
+      - name: Correct MSYS2 pthread.h to allow static libraries (otherwise you would need to use a lib DLL, rather than it being built into the EXE.)
+        shell: msys2 {0}
+        run: |
+          sed -z 's/#else\n#define WINPTHREAD_API __declspec(dllimport)/#else\n#define WINPTHREAD_API/' /${{matrix.sys}}/include/pthread.h
 
-     - name: Build ${{ matrix.build }}
-       shell: msys2 {0}
-       run: |
-         make ${{ matrix.build }}
+      - name: Build ${{ matrix.build }}
+        shell: msys2 {0}
+        run: |
+          make ${{ matrix.build }}
 
-     - name: "Upload ${{ matrix.build }} executables"
-       uses: actions/upload-artifact@v2
-       with:
-         name: windows-${{ matrix.arch }}-${{ matrix.build }}
-         path: ${{ github.workspace }}/**/tools/**/*.exe
+      - name: "Upload ${{ matrix.build }} executables"
+        uses: actions/upload-artifact@v2
+        with:
+          name: windows-${{ matrix.arch }}-${{ matrix.build }}
+          path: ${{ github.workspace }}/**/tools/**/*.exe
diff --git a/.github/workflows/build-toolchain-library-and-roms.yml b/.github/workflows/build-toolchain-library-and-roms.yml
@@ -18,6 +18,8 @@ jobs:
     steps:
       - name: Checkout
         uses: actions/checkout@v3
+        with:
+          fetch-depth: 0 # Using a full fetch so that the diff action can run.
 
       # Create a lower cased version of the repo name. This is required
       # because Docker supports only lowercase names in the registry, while