Skip to content

(GHA) Remove legacy workflows and pin SHAs - #461

Merged
Mikey Lombardi (He/Him) (michaeltlombardi) merged 2 commits into
MicrosoftDocs:mainfrom
michaeltlombardi:maint/main/gha-targets
Oct 28, 2026
Merged

Mikey Lombardi (He/Him) (michaeltlombardi) merged 2 commits into
MicrosoftDocs:mainfrom
michaeltlombardi:maint/main/gha-targets

Conversation

@michaeltlombardi

Copy link
Copy Markdown
Contributor

PR Summary

This change removes legacy workflows that are no longer supported:

  • quality.pr
  • quality.issue
  • targeting-repo-files
  • targeting-valid-branch

It pins the actions for the remaining workflows to a SHA representing the last known good commit for each action.

PR Checklist

  • Descriptive Title: This PR's title is a synopsis of the changes it proposes.
  • Summary: This PR's summary describes the scope and intent of the change.
  • Contributor's Guide: I have read the contributors guide.
  • Style: This PR adheres to the style guide.

This change removes legacy workflows that are no longer supported:

- `quality.pr`
- `quality.issue`
- `targeting-repo-files`
- `targeting-valid-branch`

It pins the actions for the remaining workflows to a SHA representing
the last known good commit for each action.
Copilot AI balanced review requested due to automatic review settings October 28, 2026 18:13
@learn-build-service-prod

Copy link
Copy Markdown

Learn Build status updates of commit 95038cd:

✅ Validation status: passed

File Status Preview URL Details
.github/workflows/checklist.yml ✅Succeeded
.github/workflows/expectations.yml ✅Succeeded
.github/workflows/quality.issue.yml ✅Succeeded n/a (file deleted or renamed)
.github/workflows/quality.pr.yml ✅Succeeded n/a (file deleted or renamed)
.github/workflows/stale-content.yml ✅Succeeded
.github/workflows/targeting-repo-files.yml ✅Succeeded n/a (file deleted or renamed)
.github/workflows/targeting-valid-branch.yml ✅Succeeded n/a (file deleted or renamed)

For more details, please refer to the build report.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The action ref in .github/workflows/expectations.yml has a trailing s, making the intended commit pin invalid and causing action resolution to fail unless a matching branch or tag exists.

1 open issue
What changed in this PR

This PR simplifies repository automation by removing four unsupported workflows and replacing mutable action references with commit pins in the remaining three workflows.

Changes:

  • Removes legacy quality-contribution labeling and assignment workflows.
  • Removes authorization workflows for repository-file changes and PRs targeting live.
  • Updates checklist, expectations, and stale-content action references from main to commit-based refs.
File Description
.github/​workflows/​targeting-valid-branch.yml Removes the live-branch authorization workflow.
.github/​workflows/​targeting-repo-files.yml Removes the repository-file authorization workflow.
.github/​workflows/​stale-content.yml Pins the stale-content reporting action.
.github/​workflows/​quality.pr.yml Removes quality-contribution PR labeling.
.github/​workflows/​quality.issue.yml Removes quality-issue assignment and labeling.
.github/​workflows/​expectations.yml Changes the community-PR guidance action ref to an intended commit pin.
.github/​workflows/​checklist.yml Pins the checklist verification action.

Review effort: Balanced

Comment thread .github/workflows/expectations.yml Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@learn-build-service-prod

Copy link
Copy Markdown

Learn Build status updates of commit 791eb9a:

✅ Validation status: passed

File Status Preview URL Details
.github/workflows/checklist.yml ✅Succeeded
.github/workflows/expectations.yml ✅Succeeded
.github/workflows/quality.issue.yml ✅Succeeded n/a (file deleted or renamed)
.github/workflows/quality.pr.yml ✅Succeeded n/a (file deleted or renamed)
.github/workflows/stale-content.yml ✅Succeeded
.github/workflows/targeting-repo-files.yml ✅Succeeded n/a (file deleted or renamed)
.github/workflows/targeting-valid-branch.yml ✅Succeeded n/a (file deleted or renamed)

For more details, please refer to the build report.

@michaeltlombardi
Mikey Lombardi (He/Him) (michaeltlombardi) merged commit 9cbf4d9 into MicrosoftDocs:main Oct 28, 2026
4 of 5 checks passed
@michaeltlombardi
Mikey Lombardi (He/Him) (michaeltlombardi) deleted the maint/main/gha-targets branch October 28, 2026 18:23
@learn-build-service-prod

Copy link
Copy Markdown

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

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