Skip to content

trusted-fork-NeethuESim/pr-236-ac7e427658bcf31ec2092c851c8121c9694e4b0d - #253

Merged
JoeStech merged 8 commits into
mainfrom
trusted-fork-NeethuESim/pr-236-ac7e427658bcf31ec2092c851c8121c9694e4b0d
Oct 23, 2026
Merged

JoeStech merged 8 commits into
mainfrom
trusted-fork-NeethuESim/pr-236-ac7e427658bcf31ec2092c851c8121c9694e4b0d

Conversation

@arm-mcp-build

@arm-mcp-build arm-mcp-build Bot commented Oct 23, 2026

Copy link
Copy Markdown
Contributor

Promotes the exact revision from a fork pull request into an internal branch.

After this internal pull request merges, close the original pull request manually.

The required Black Duck guard accepts this branch only while its head remains at the reviewed commit. A changed fork head requires a new promotion.

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.

🟢 Approval recommended

The documentation and generated-message changes align with the existing promotion workflow and introduce no identified blocking issues.

No open issues 🎉

What changed in this PR

This PR clarifies the fork-promotion process for Black Duck checks and adds reviewer guidance to generated internal pull requests. It does not change promotion logic.

Changes:

  • Documents manual promotion requirements and workflow logging for untrusted fork pull requests.
  • Directs reviewers to discuss changes and push commits on the original fork pull request.
File Description
docs/​fork-pr-promotion.md Clarifies manual promotion steps and Black Duck check expectations.
.github/​scripts/​promote-fork-pr.py Adds guidance to the generated internal PR description.

Review effort: Balanced

@JoeStech
JoeStech merged commit 2368fc3 into main Oct 23, 2026
22 of 23 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