Skip to content

fix: fix merge-approved-pr action path for cross-repo workflow calls#224

Merged
chloeYue merged 1 commit intomainfrom
fix-merge-my-pr-flow
Mar 16, 2026
Merged

fix: fix merge-approved-pr action path for cross-repo workflow calls#224
chloeYue merged 1 commit intomainfrom
fix-merge-my-pr-flow

Conversation

@chloeYue
Copy link
Contributor

@chloeYue chloeYue commented Mar 16, 2026

Problem

When metamask-mobile (or any external repo) calls this reusable workflow via workflow_call, the relative path:

uses: ./.github/actions/merge-approved-pr

resolves against the calling repository, not github-tools where the workflow is defined. GitHub looks for metamask-mobile/.github/actions/merge-approved-pr, which doesn't exist — causing the "Merge my PR" flow to silently fail.

Example failing PR: MetaMask/metamask-mobile#27467

Fix

Replace the relative path with an absolute ref:

uses: MetaMask/github-tools/.github/actions/merge-approved-pr@v1

This ensures the action is always resolved from github-tools regardless of which repo triggers the reusable workflow.

When metamask-mobile calls this reusable workflow, the relative path
`./.github/actions/merge-approved-pr` resolves to the calling repo
(metamask-mobile) instead of github-tools, causing the workflow to fail.

Replace with an absolute ref so the action is always resolved correctly.

Made-with: Cursor
@chloeYue chloeYue requested a review from a team March 16, 2026 11:15
@chloeYue chloeYue self-assigned this Mar 16, 2026
@chloeYue chloeYue requested review from Gudahtt and HowardBraham and removed request for a team March 16, 2026 11:15
@chloeYue chloeYue changed the title fix: resolve merge-approved-pr action path for cross-repo workflow calls fix: fix merge-approved-pr action path for cross-repo workflow calls Mar 16, 2026
@chloeYue chloeYue merged commit 262de41 into main Mar 16, 2026
21 checks passed
@chloeYue chloeYue deleted the fix-merge-my-pr-flow branch March 16, 2026 12:17
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