update board check

Author: tannewt

.github/workflows/build.yml

@@ -66,7 +66,7 @@ jobs:
     - name: Translations
       run: make check-translate
     - name: New boards check
-      run: python3 -u travis_new_boards_check.py
+      run: python3 -u ci_new_boards_check.py
       working-directory: tools
 
   build-arm:
@@ -76,77 +76,77 @@ jobs:
       fail-fast: false
       matrix:
         board:
-        - "metro_m0_express"
-        - "metro_m4_express"
-        - "trinket_m0_haxpress"
-        - "circuitplayground_express"
-        - "mini_sam_m4"
-        - "grandcentral_m4_express"
+        - "arduino_mkr1300"
+        - "arduino_mkrzero"
+        - "arduino_zero"
+        - "bast_pro_mini_m0"
         - "capablerobot_usbhub"
-        - "pygamer"
-        - "pca10056"
-        - "pca10059"
-        - "feather_nrf52840_express"
-        - "metro_nrf52840_express"
+        - "catwan_usbstick"
         - "circuitplayground_bluefruit"
-        - "makerdiary_nrf52840_mdk"
-        - "makerdiary_nrf52840_mdk_usb_dongle"
-        - "particle_boron"
-        - "particle_argon"
-        - "particle_xenon"
-        - "sparkfun_nrf52840_mini"
-        - "electronut_labs_papyr"
-        - "electronut_labs_blip"
-        - "metro_m4_airlift_lite"
-        - "pirkey_m0"
-        - "trellis_m4_express"
-        - "trinket_m0"
-        - "sparkfun_lumidrive"
-        - "sparkfun_redboard_turbo"
-        - "bast_pro_mini_m0"
-        - "datum_distance"
-        - "pyruler"
+        - "circuitplayground_express"
+        - "circuitplayground_express_crickit"
         - "cp32-m4"
+        - "datalore_ip_m4"
+        - "datum_distance"
+        - "datum_imu"
+        - "datum_light"
+        - "datum_weather"
+        - "electronut_labs_blip"
+        - "electronut_labs_papyr"
+        - "escornabot_makech"
+        - "feather_m0_adalogger"
+        - "feather_m0_basic"
+        - "feather_m0_express"
+        - "feather_m0_express_crickit"
+        - "feather_m0_rfm69"
+        - "feather_m0_rfm9x"
+        - "feather_m0_supersized"
+        - "feather_m4_express"
+        - "feather_nrf52840_express"
         - "feather_radiofruit_zigbee"
         - "gemma_m0"
+        - "grandcentral_m4_express"
         - "hallowing_m0_express"
         - "itsybitsy_m0_express"
         - "itsybitsy_m4_express"
+        - "kicksat-sprite"
+        - "makerdiary_nrf52840_mdk"
+        - "makerdiary_nrf52840_mdk_usb_dongle"
         - "meowmeow"
-        - "sam32"
-        - "uchip"
-        - "escornabot_makech"
-        - "pygamer_advance"
-        - "datum_imu"
-        - "snekboard"
-        - "feather_m0_supersized"
-        - "feather_m0_express_crickit"
-        - "feather_m0_rfm69"
-        - "feather_m0_rfm9x"
-        - "feather_m4_express"
-        - "arduino_zero"
-        - "arduino_mkr1300"
-        - "arduino_mkrzero"
+        - "metro_m0_express"
+        - "metro_m4_airlift_lite"
+        - "metro_m4_express"
+        - "metro_nrf52840_express"
+        - "mini_sam_m4"
+        - "particle_argon"
+        - "particle_boron"
+        - "particle_xenon"
+        - "pca10056"
+        - "pca10059"
         - "pewpew10"
-        - "kicksat-sprite"
-        - "ugame10"
-        - "robohatmm1_m0"
-        - "robohatmm1_m4"
-        - "datum_light"
-        - "datalore_ip_m4"
-        - "circuitplayground_express_crickit"
-        - "feather_m0_adalogger"
-        - "feather_m0_basic"
-        - "feather_m0_express"
-        - "catwan_usbstick"
-        - "pyportal"
-        - "sparkfun_samd21_mini"
-        - "sparkfun_samd21_dev"
+        - "pirkey_m0"
         - "pybadge"
         - "pybadge_airlift"
-        - "datum_weather"
+        - "pygamer"
+        - "pygamer_advance"
+        - "pyportal"
         - "pyportal_titano"
-    
+        - "pyruler"
+        - "robohatmm1_m0"
+        - "robohatmm1_m4"
+        - "sam32"
+        - "snekboard"
+        - "sparkfun_lumidrive"
+        - "sparkfun_nrf52840_mini"
+        - "sparkfun_redboard_turbo"
+        - "sparkfun_samd21_dev"
+        - "sparkfun_samd21_mini"
+        - "trellis_m4_express"
+        - "trinket_m0"
+        - "trinket_m0_haxpress"
+        - "uchip"
+        - "ugame10"
+
     steps:
     - name: Set up Python 3.5
       uses: actions/setup-python@v1
@@ -173,6 +173,7 @@ jobs:
 
   website:
     runs-on: ubuntu-16.04
+    needs: build-arm
     steps:
     - name: Set up Python 3.5
       uses: actions/setup-python@v1

tools/ci_new_boards_check.py

@@ -0,0 +1,50 @@
+#! /usr/bin/env python3
+
+import sys
+import os
+import json
+import yaml
+
+import build_board_info
+
+workflow_file = '.github/workflows/build.yml'
+
+# Get boards in json format
+boards_info_json = build_board_info.get_board_mapping()
+
+# Get all the boards out of the json format
+info_boards = [board for board in boards_info_json.keys() if not boards_info_json[board].get("alias", False)]
+
+# We need to know the path of the workflow file
+base_path = os.path.dirname(__file__)
+yml_path = os.path.abspath(os.path.join(base_path, '..', workflow_file))
+
+# Loading board list based on build jobs in the workflow file.
+ci_boards = []
+with open(yml_path, "r") as f:
+    workflow = yaml.safe_load(f)
+
+ok = True
+for job in workflow["jobs"]:
+    if not job.startswith("build"):
+        continue
+    job_boards = workflow["jobs"][job]["strategy"]["matrix"]["board"]
+    if job_boards != sorted(job_boards):
+        print("Boards for job \"{}\" not sorted. Must be:".format(job))
+        print("        - \"" + "\"\n        - \"".join(sorted(job_boards)) + "\"")
+        ok = False
+    ci_boards.extend(job_boards)
+
+# All the travis_boards elements must be on info_boards
+info_boards.sort()
+ci_boards.sort()
+
+missing_boards = set(info_boards) - set(ci_boards)
+
+if missing_boards:
+    print('Boards missing in {}:'.format(workflow_file))
+    for board in missing_boards:
+        print(board)
+
+if not ok:
+    sys.exit(1)