Skip to content

introduces change detection job and a "ready to merge" running on Linux to avoid macOS runner congestion#1270

Merged
TravisEz13 merged 7 commits intomasterfrom
TravisEz13-patch-run-chagedetection-on-linux-from-masterv2
Mar 27, 2025
Merged

introduces change detection job and a "ready to merge" running on Linux to avoid macOS runner congestion#1270
TravisEz13 merged 7 commits intomasterfrom
TravisEz13-patch-run-chagedetection-on-linux-from-masterv2

Conversation

@TravisEz13
Copy link
Copy Markdown
Member

@TravisEz13 TravisEz13 commented Mar 27, 2025

PR Summary

This pull request introduces a new change detection job and a "ready to merge" job to several GitHub workflow files. The change detection job identifies if specific files have been modified, and the "ready to merge" job ensures that the changes are ready for merging after successful completion of other jobs.

New Change Detection Job:

  • Added a changes job to detect if specific files have been modified, using the dorny/paths-filter action. This job runs on ubuntu-latest and sets outputs based on the detected changes. (.github/workflows/PR-powershell-daily.yml, [1]; .github/workflows/PR-powershell-lts.yml, [2]; .github/workflows/PR-powershell-preview.yml, [3]; .github/workflows/PR-powershell.yml, [4]

New Ready to Merge Job:

  • Added a ready_to_merge job that depends on the completion of other jobs and uses the PowerShell/compliance/.github/workflows/ready-to-merge.yml action. This job runs unconditionally (if: always()) and ensures that the changes are ready for merging. (.github/workflows/PR-powershell-daily.yml, [1]; .github/workflows/PR-powershell-lts.yml, Fc269bdfL51R82; .github/workflows/PR-powershell-preview.yml, [2]; .github/workflows/PR-powershell.yml, [3]

Integration with Existing Jobs:

  • Modified existing jobs (homebrew-formula-daily, homebrew-formula-lts, homebrew-formula-preview, homebrew-formula-stable) to depend on the new changes job and only run if the relevant files have been modified. (.github/workflows/PR-powershell-daily.yml, [1]; .github/workflows/PR-powershell-lts.yml, [2]; .github/workflows/PR-powershell-preview.yml, [3]; .github/workflows/PR-powershell.yml, [4]

PR Context

Comment thread .github/workflows/PR-powershell-lts.yml Dismissed
Comment thread .github/workflows/PR-powershell-lts.yml Outdated
Comment thread .github/workflows/PR-powershell-preview.yml Dismissed
Comment thread .github/workflows/PR-powershell.yml Dismissed
Comment thread .github/workflows/PR-powershell-daily.yml Dismissed
@TravisEz13 TravisEz13 closed this Mar 27, 2025
@TravisEz13 TravisEz13 reopened this Mar 27, 2025
@TravisEz13 TravisEz13 changed the title Update PR-powershell-lts.yml introduces change detection job and a "ready to merge" running on Linux to avoid macOS runner congestion Mar 27, 2025
@TravisEz13 TravisEz13 marked this pull request as ready for review March 27, 2025 18:12
@TravisEz13 TravisEz13 merged commit 9e1b43f into master Mar 27, 2025
28 checks passed
@TravisEz13 TravisEz13 deleted the TravisEz13-patch-run-chagedetection-on-linux-from-masterv2 branch March 27, 2025 18:14
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.

2 participants