From b75c9a84743e839eff8145e8751dfc663e614675 Mon Sep 17 00:00:00 2001
From: Lee Lup Yuen <luppy@appkaki.com>
Date: Tue, 28 Jul 2020 10:27:38 +0800
Subject: [PATCH] Clean up

---
 .github/workflows/main2.yml | 27 ++++++++++++++++++++++-----
 1 file changed, 22 insertions(+), 5 deletions(-)

diff --git a/.github/workflows/main2.yml b/.github/workflows/main2.yml
index c8ad3770..b54f1250 100644
--- a/.github/workflows/main2.yml
+++ b/.github/workflows/main2.yml
@@ -1,6 +1,7 @@
 # GitHub Actions Workflow to build FreeRTOS Firmware for PineTime Smart Watch
 # See https://lupyuen.github.io/pinetime-rust-mynewt/articles/cloud
 # Based on https://github.com/JF002/Pinetime/blob/master/doc/buildAndProgram.md
+# and https://github.com/JF002/Pinetime/blob/master/bootloader/README.md
 
 # Name of this Workflow
 name: Build PineTime Firmware
@@ -24,6 +25,10 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
+
+    #########################################################################################
+    # Download and Cache Dependencies
+
     - name: Install cmake
       uses: lukka/get-cmake@v3.18.0
 
@@ -60,39 +65,51 @@ jobs:
       if:   steps.cache-nrf5sdk.outputs.cache-hit != 'true'  # Install SDK if not found in cache
       run:  cd ${{ runner.temp }} && curl https://developer.nordicsemi.com/nRF5_SDK/nRF5_SDK_v15.x.x/nRF5_SDK_15.3.0_59ac345.zip -o nrf5_sdk.zip && unzip nrf5_sdk.zip && mv nRF5_SDK_15.3.0_59ac345 nrf5_sdk
 
+    #########################################################################################
+    # Checkout
+      
     - name: Checkout source files
       uses: actions/checkout@v2
 
     - name: Show files
       run:  set ; pwd ; ls -l
 
+    #########################################################################################
+    # CMake
+
     - name: CMake
       run:  mkdir -p build && cd build && cmake -DARM_NONE_EABI_TOOLCHAIN_PATH=${{ runner.temp }}/arm-none-eabi -DNRF5_SDK_PATH=${{ runner.temp }}/nrf5_sdk -DUSE_OPENOCD=1 ../
       
+    #########################################################################################
+    # Make and upload pinetime-mcuboot-app
+    # For Debugging Builds: Remove "make" option "-j" for clearer output. Add "--trace" to see details.
+    # For Faster Builds: Add "make" option "-j"
+      
     - name: Make pinetime-mcuboot-app
       run:  cd build && make pinetime-mcuboot-app
 
     - name: Upload pinetime-mcuboot-app
       uses: actions/upload-artifact@v2
       with:
-        # Artifact name (optional)
         name: pinetime-mcuboot-app.bin
         # A file, directory or wildcard pattern that describes what to upload
         path: build/src/pinetime-mcuboot-app.bin
 
+    #########################################################################################
+    # Make and upload pinetime-app
+      
     - name: Make pinetime-app
-      # For Debugging Builds: Remove "make" option "-j" for clearer output. Add "--trace" to see details.
-      # For Faster Builds: Add "make" option "-j"
       run:  cd build && make pinetime-app
 
     - name: Upload pinetime-app
       uses: actions/upload-artifact@v2
       with:
-        # Artifact name (optional)
         name: pinetime-app.out
-        # A file, directory or wildcard pattern that describes what to upload
         path: build/src/pinetime-app.out
 
+    #########################################################################################
+    # Finish
+
     - name: Find output
       run:  find . -name "pinetime-app.*" -ls ; find . -name "pinetime-mcuboot-app.*" -ls