-
Notifications
You must be signed in to change notification settings - Fork 7.8k
Update GitHub Actions to work in private GitHub repo #25197
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this 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 pull request updates several GitHub Actions workflows so that they run only when the repository owner meets specific criteria and updates workflow references from the default branch name to a versioned tag. The changes ensure that workflows execute only in appropriate repository contexts and reference a specific release version rather than the master branch.
Reviewed Changes
Copilot reviewed 7 out of 8 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
.github/workflows/macos-ci.yml | Added owner condition to the change detection job and updated ready-to-merge reference. |
.github/workflows/windows-ci.yml | Added owner condition to the change detection job and updated ready-to-merge reference. |
.github/workflows/linux-ci.yml | Added owner condition to the change detection job and updated ready-to-merge reference. |
.github/workflows/AssignPrs.yml | Added an if condition to restrict execution based on repository owner. |
.github/workflows/scorecards.yml | Added an if condition to restrict execution based on repository owner. |
.github/workflows/processReminders.yml | Added an if condition to restrict execution based on repository owner. |
.github/workflows/markdownLink.yml | Added an if condition to restrict execution based on repository owner. |
Files not reviewed (1)
- .github/workflows/backport.yml: Language not supported
/azp run PowerShell-CI-linux-packaging, PowerShell-Windows-Packaging-CI |
Azure Pipelines could not run because the pipeline triggers exclude this branch/path. |
📣 Hey @TravisEz13, how did we do? We would love to hear your feedback with the link below! 🗣️ 🔗 https://aka.ms/PSRepoFeedback |
This pull request includes several changes to the GitHub Actions workflows to improve conditional execution and update workflow references. The most important changes include adding conditional checks based on the repository owner and updating workflow references to specific versions.
Conditional execution based on repository owner:
.github/workflows/AssignPrs.yml
: Added a condition to execute the job only if the repository owner is 'PowerShell'..github/workflows/linux-ci.yml
: Added conditions to execute jobs only if the repository owner starts with 'azure' or is 'PowerShell'..github/workflows/macos-ci.yml
: Added conditions to execute jobs only if the repository owner starts with 'azure' or is 'PowerShell'..github/workflows/windows-ci.yml
: Added conditions to execute jobs only if the repository owner starts with 'azure' or is 'PowerShell'..github/workflows/markdownLink.yml
: Added a condition to execute the job only if the repository owner is 'PowerShell'..github/workflows/processReminders.yml
: Added a condition to execute the job only if the repository owner is 'PowerShell'..github/workflows/scorecards.yml
: Added a condition to execute the job only if the repository owner is 'PowerShell'.Updating workflow references:
.github/workflows/linux-ci.yml
: Updated the reference for theready-to-merge
workflow frommaster
tov1.0.0
..github/workflows/macos-ci.yml
: Updated the reference for theready-to-merge
workflow frommaster
tov1.0.0
..github/workflows/windows-ci.yml
: Updated the reference for theready-to-merge
workflow frommaster
tov1.0.0
.Additionally, the
.github/workflows/backport.yml
file has been removed entirely.