Skip to content

workflows: remove per-commit checking#22

Merged
kurtjd merged 1 commit intoOpenDevicePartnership:mainfrom
jerrysxie:remove-per-commit-checking
Jun 19, 2025
Merged

workflows: remove per-commit checking#22
kurtjd merged 1 commit intoOpenDevicePartnership:mainfrom
jerrysxie:remove-per-commit-checking

Conversation

@jerrysxie
Copy link
Contributor

No description provided.

@jerrysxie jerrysxie requested a review from kurtjd June 19, 2025 19:55
@jerrysxie jerrysxie marked this pull request as ready for review June 19, 2025 19:55
Copilot AI review requested due to automatic review settings June 19, 2025 19:55
@jerrysxie jerrysxie requested a review from a team as a code owner June 19, 2025 19:55
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR removes per-commit checking by deleting the commit_list job and all matrix.commit references, simplifying job names and checkout steps.

  • Deleted the commit_list job and related outputs from check.yml
  • Removed needs: commit_list and matrix.commit usages in all jobs
  • Simplified job name fields (dropped commit hash suffix) in both workflows

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/rolling.yml Dropped ${{ matrix.commit }} from MSRV job name
.github/workflows/check.yml Removed commit_list job, eliminated per-commit matrices

@kurtjd kurtjd merged commit 381cbe8 into OpenDevicePartnership:main Jun 19, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants