Merge branch 'OpenDevicePartnership:main' into espi_refactor

Author: philgweber

.github/workflows/cargo-vet.yml

@@ -10,47 +10,27 @@ concurrency:
 
 name: cargo-vet
 jobs:
-  commit_list:
-    uses: ./.github/workflows/commit-list.yml
-    with:
-      event_name: ${{ github.event_name }}
-      base_ref: ${{ github.base_ref }}
-      head_sha: ${{ github.event.pull_request.head.sha }}
-
   vet:
     # cargo-vet checks for unvetted dependencies in the Cargo.lock file
     # This is to ensure that new dependencies are vetted before they are added to the project
     name: vet-dependencies
     runs-on: ubuntu-latest
-    needs: commit_list
     env:
       CARGO_VET_VERSION: 0.10.1
-    
-    strategy:
-      fail-fast: false
-      matrix:
-        commit: ${{ fromJSON(needs.commit_list.outputs.commits) }}
-    
     steps:
     - uses: actions/checkout@v4
       with:
         submodules: true
-        ref: ${{ matrix.commit }}
-  
     - uses: actions/cache@v4
       with:
         path: ${{ runner.tool_cache }}/cargo-vet
         key: cargo-vet-bin-${{ env.CARGO_VET_VERSION }}
-  
     - name: Add the tool cache directory to the search path
       run: echo "${{ runner.tool_cache }}/cargo-vet/bin" >> $GITHUB_PATH
-  
     - name: Ensure that the tool cache is populated with the cargo-vet binary
       run: cargo install --root ${{ runner.tool_cache }}/cargo-vet --version ${{ env.CARGO_VET_VERSION }} cargo-vet
-  
     - name: Invoke cargo-vet
       run: cargo vet --locked
-    
     - name: Save PR number
     # PR number is saved as an artifact so it can be used to determine the PR to comment on by the vet-pr-comment workflow
     # vet-pr-comment workflow is triggered by the workflow_run event so it runs in the context of the base branch and not the PR branch

.github/workflows/check.yml

@@ -1,6 +1,5 @@
 # This workflow runs whenever a PR is opened or updated, or a commit is pushed to main. It runs
 # several checks:
-# - commit_list: produces a list of commits to be checked
 # - fmt: checks that the code is formatted according to rustfmt
 # - clippy: checks that the code does not contain any clippy warnings
 # - doc: checks that the code can be documented without errors
@@ -23,26 +22,13 @@ concurrency:
 name: check
 jobs:
 
-  commit_list:
-    uses: ./.github/workflows/commit-list.yml
-    with:
-      event_name: ${{ github.event_name }}
-      base_ref: ${{ github.base_ref }}
-      head_sha: ${{ github.event.pull_request.head.sha || github.sha }}
-
   fmt:
     runs-on: ubuntu-latest
     name: stable / fmt
-    needs: commit_list
-    strategy:
-      fail-fast: false
-      matrix:
-        commit: ${{ fromJSON(needs.commit_list.outputs.commits) }}
     steps:
       - uses: actions/checkout@v4
         with:
           submodules: true
-          ref: ${{ matrix.commit }}
       - name: Install stable
         uses: dtolnay/rust-toolchain@stable
         with:
@@ -53,7 +39,6 @@ jobs:
   clippy:
     runs-on: ubuntu-latest
     name: ${{ matrix.toolchain }} / clippy
-    needs: commit_list
     permissions:
       contents: read
       checks: write
@@ -62,12 +47,10 @@ jobs:
       matrix:
         # Get early warning of new lints which are regularly introduced in beta channels.
         toolchain: [stable, beta]
-        commit: ${{ fromJSON(needs.commit_list.outputs.commits) }}
     steps:
       - uses: actions/checkout@v4
         with:
           submodules: true
-          ref: ${{ matrix.commit }}
       - name: Install ${{ matrix.toolchain }}
         uses: dtolnay/rust-toolchain@master
         with:
@@ -83,16 +66,10 @@ jobs:
   # semver:
   #   runs-on: ubuntu-latest
   #   name: semver
-  #   needs: commit_list
-  #   strategy:
-  #     fail-fast: false
-  #     matrix:
-  #       commit: ${{ fromJSON(needs.commit_list.outputs.commits) }}
   #   steps:
   #     - uses: actions/checkout@v4
   #       with:
   #         submodules: true
-  #         ref: ${{ matrix.commit }}
   #     - name: Install stable
   #       uses: dtolnay/rust-toolchain@stable
   #       with:
@@ -106,16 +83,10 @@ jobs:
     # API be documented as only available in some specific platforms.
     runs-on: ubuntu-latest
     name: nightly / doc
-    needs: commit_list
-    strategy:
-      fail-fast: false
-      matrix:
-        commit: ${{ fromJSON(needs.commit_list.outputs.commits) }}
     steps:
       - uses: actions/checkout@v4
         with:
           submodules: true
-          ref: ${{ matrix.commit }}
       - name: Install nightly
         uses: dtolnay/rust-toolchain@nightly
       - name: cargo doc
@@ -130,16 +101,12 @@ jobs:
     # which is required for feature unification
     runs-on: ubuntu-latest
     name: ubuntu / stable / features
-    needs: commit_list
     strategy:
       fail-fast: false
-      matrix:
-        commit: ${{ fromJSON(needs.commit_list.outputs.commits) }}
     steps:
       - uses: actions/checkout@v4
         with:
           submodules: true
-          ref: ${{ matrix.commit }}
       - name: Install stable
         uses: dtolnay/rust-toolchain@stable
       - name: cargo install cargo-hack
@@ -154,16 +121,10 @@ jobs:
     # our dependencies.
     runs-on: ubuntu-latest
     name: ubuntu / stable / deny
-    needs: commit_list
-    strategy:
-      fail-fast: false
-      matrix:
-        commit: ${{ fromJSON(needs.commit_list.outputs.commits) }}
     steps:
       - uses: actions/checkout@v4
         with:
           submodules: true
-          ref: ${{ matrix.commit }}
       - name: Install stable
         uses: dtolnay/rust-toolchain@stable
       - name: cargo install cargo-deny
@@ -176,16 +137,10 @@ jobs:
   test:
     runs-on: ubuntu-latest
     name: ubuntu / stable / test
-    needs: commit_list
-    strategy:
-      fail-fast: false
-      matrix:
-        commit: ${{ fromJSON(needs.commit_list.outputs.commits) }}
     steps:
       - uses: actions/checkout@v4
         with:
           submodules: true
-          ref: ${{ matrix.commit }}
       - name: Install stable
         uses: dtolnay/rust-toolchain@stable
       - name: cargo test
@@ -194,20 +149,17 @@ jobs:
   msrv:
     # check that we can build using the minimal rust version that is specified by this crate
     runs-on: ubuntu-latest
-    needs: commit_list
     # we use a matrix here just because env can't be used in job names
     # https://docs.github.com/en/actions/learn-github-actions/contexts#context-availability
     strategy:
       fail-fast: false
       matrix:
-        commit: ${{ fromJSON(needs.commit_list.outputs.commits) }}
         msrv: ["1.85"]
     name: ubuntu / ${{ matrix.msrv }}
     steps:
       - uses: actions/checkout@v4
         with:
           submodules: true
-          ref: ${{ matrix.commit }}
       - name: Install ${{ matrix.msrv }}
         uses: dtolnay/rust-toolchain@master
         with:
@@ -220,20 +172,17 @@ jobs:
 
   check-arm-examples:
     runs-on: ubuntu-latest
-    needs: commit_list
     # we use a matrix here just because env can't be used in job names
     # https://docs.github.com/en/actions/learn-github-actions/contexts#context-availability
     strategy:
       fail-fast: false
       matrix:
-        commit: ${{ fromJSON(needs.commit_list.outputs.commits) }}
         example_directory: ["examples/rt633", "examples/rt685s-evk"]
     name: ubuntu / check-examples
     steps:
       - uses: actions/checkout@v4
         with:
           submodules: true
-          ref: ${{ matrix.commit }}
       - name: Install stable
         uses: dtolnay/rust-toolchain@stable
       - name: cargo check
@@ -243,20 +192,17 @@ jobs:
 
   check-std-examples:
     runs-on: ubuntu-latest
-    needs: commit_list
     # we use a matrix here just because env can't be used in job names
     # https://docs.github.com/en/actions/learn-github-actions/contexts#context-availability
     strategy:
       fail-fast: false
       matrix:
-        commit: ${{ fromJSON(needs.commit_list.outputs.commits) }}
         example_directory: ["examples/std"]
     name: ubuntu / check-examples
     steps:
       - uses: actions/checkout@v4
         with:
           submodules: true
-          ref: ${{ matrix.commit }}
       - name: Install stable
         uses: dtolnay/rust-toolchain@stable
       - name: cargo check