Skip to content

workflows: remove per-commit checking#22

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

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

Conversation

@jerrysxie

Copy link
Copy Markdown
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

Copilot AI left a comment

Copy link
Copy Markdown

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

None yet

Development

Successfully merging this pull request may close these issues.

4 participants