Skip to content

Conversation

@Taojunshen
Copy link
Contributor

Prior to this change, the GHA workflow for checking authorization on submitting PRs to the live branch didn't include the authorized_accounts parameter.

When you retrieve the permissions for the learn-build-service-prod[bot] account, the API reports that the automation account lacks all permissions.

This change adds the bot account to an allow-list to prevent the error.

PR Summary

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.

Prior to this change, the GHA workflow for checking authorization on submitting PRs to the `live` branch didn't include the `authorized_accounts` parameter.

When you retrieve the permissions for the `learn-build-service-prod[bot]` account, the API reports that the automation account lacks all permissions.

This change adds the bot account to an allow-list to prevent the error.
@learn-build-service-prod
Copy link
Contributor

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.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 78459b4:

✅ Validation status: passed

File Status Preview URL Details
.github/workflows/targeting-valid-branch.yml ✅Succeeded

For more details, please refer to the build report.

@Taojunshen Taojunshen merged commit c941178 into live Jan 14, 2026
14 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.

3 participants