Add Python autoformatter (#513)

* Add Python autoformatter (no actual formatting yet)

* Don't need a venv!

* Run formatter!

* Run formatter on everything else

* Remove --verbose

* Some cleanups

* Reformat with magic trailing comma skipped

Author: multiplemonomials

.github/workflows/basic_checks.yml

@@ -86,26 +86,25 @@ jobs:
 
   style-check:
     runs-on: ubuntu-latest
-    container:
-      image: ghcr.io/armmbed/mbed-os-env:master-latest
-
     steps:
 
       - name: Checkout repo
-        uses: actions/checkout@v4
+        uses: actions/checkout@v5
         with:
           fetch-depth: 0
+
+      - name: Install Formatters
+        run: |
+          sudo apt-get update -y
+          sudo apt-get install astyle aspell
     
-      - 
-        name: UTF-8 Check
+      - name: UTF-8 Check
         run: |
           git config --global --add safe.directory "$GITHUB_WORKSPACE"
           # Make sure we're not introducing any text which is not UTF-8 encoded
           git diff origin/${GITHUB_BASE_REF} -U0 | ( grep -a '^+' || true ) | ( ! grep -axv '.*' )
-          
 
-      - 
-        name: astyle checks
+      - name: astyle checks
         run: |
           set -x
           git config --global --add safe.directory "$GITHUB_WORKSPACE"
@@ -134,11 +133,26 @@ jobs:
           tee BUILD/badlibs |
           sed -e "s/^/Bad library name found: /" && [ ! -s BUILD/badlibs ]
           # Assert that all assembler files are named correctly
-          # The strange command below asserts that there are exactly 0 libraries
+          # The strange command below asserts that there are exactly 0 files
           # that do end with .s
           find  -name "*.s" | tee BUILD/badasm |
           sed -e "s/^/Bad Assembler file name found: /" && [ ! -s BUILD/badasm ]
 
+      # Set up the oldest python version that's currently still supported by Python (as of Fall 2025)
+      - uses: actions/setup-python@v6
+        with:
+          python-version: '3.9'
+
+      - name: Install Python packages
+        run: |
+          pip install -e ./tools[linters]
+
+      - name: Check Python Formatting
+        run: |
+          cd tools
+          ruff format --diff
+          
+
   docs-check:
     runs-on: ubuntu-latest
 
@@ -162,46 +176,39 @@ jobs:
   python-tests:
     runs-on: ubuntu-latest
     steps:
-      -
-        name: Checkout repo
-        uses: actions/checkout@v4
-        
-      - name: Install python3-venv
-        run: |
-          sudo apt-get update
-          sudo apt-get install -y python3-venv
+      - name: Checkout repo
+        uses: actions/checkout@v5
+
+      # Set up the oldest python version that's currently still supported by Python (as of Fall 2025)
+      - uses: actions/setup-python@v6
+        with:
+          python-version: '3.9'
 
       - name: Install Python packages
         run: |
-          python3 -m venv venv
-          source venv/bin/activate
           pip install -e ./tools[unit-tests]
 
       - name: Python Tests
         run: |
-          source venv/bin/activate
           tools/run_python_tests.sh
 
   check-cmsis-mcu-descriptions-matches-target-list:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout repo
-        uses: actions/checkout@v4
+        uses: actions/checkout@v5
 
-      - name: Install python3-venv
-        run: |
-          sudo apt-get update
-          sudo apt-get install -y python3-venv
+      # Set up the oldest python version that's currently still supported by Python (as of Fall 2025)
+      - uses: actions/setup-python@v6
+        with:
+          python-version: '3.9'
 
       - name: Install Python packages
         run: |
-          python3 -m venv venv
-          source venv/bin/activate
           pip install -e ./tools
 
       - name: Verify that cmsis_mcu_descriptions.json5 is in sync with targets.json5
         run: |
-          source venv/bin/activate
           cd tools/python
           mbed-tools cmsis-mcu-descr find-unused
           mbed-tools cmsis-mcu-descr check-missing
@@ -214,7 +221,7 @@ jobs:
     steps:
       -
         name: Checkout repo
-        uses: actions/checkout@v4
+        uses: actions/checkout@v5
         with:
           fetch-depth: 0
 

tools/cmake/upload_methods/python_packagecheck.py

@@ -1,7 +1,7 @@
 # Copyright (c) 2020 ARM Limited. All rights reserved.
 # SPDX-License-Identifier: Apache-2.0
 
-#file which is invoked by the cmake build system to check if all necessary python packages are installed.
+# file which is invoked by the cmake build system to check if all necessary python packages are installed.
 
 import sys
 
@@ -10,4 +10,4 @@
 except ImportError:
     exit(1)
 
-exit(0)
+exit(0)