Skip to content

Fix workflows: upstream renamed facebook/react → react/react - #557

Open
mahdyaralipor wants to merge 2 commits into
reactjs:mainfrom
mahdyaralipor:fix-label-workflow-upstream-rename
Open

Fix workflows: upstream renamed facebook/react → react/react#557
mahdyaralipor wants to merge 2 commits into
reactjs:mainfrom
mahdyaralipor:fix-label-workflow-upstream-rename

Conversation

@mahdyaralipor

@mahdyaralipor mahdyaralipor commented Oct 4, 2026 •
edited
Loading

Copy link
Copy Markdown

دو ورک‌فلو (Label Core Team PRs و Discord Notify) به facebook/react ارجاع می‌دادند، ولی ریپوی اصلی به react/react منتقل شده و گیت‌هاب اسم قدیمی را برای uses: دنبال نمی‌کند. نتیجه: خطای workflow was not found روی همه PRها.\n\nاین PR هر دو ارجاع را به اسم جدید تغییر می‌دهد. فایل مقصد (shared_check_maintainer.yml@main) سالم است و تریگر workflow_call دارد.\n\nبقیه موارد facebook/react در ریپو فقط لینک https معمولی‌اند (ریدایرکت مرورگر کار می‌کند) و پارس Actions را خراب نمی‌کنند.

@vercel

vercel Bot commented Oct 4, 2026 •
edited
Loading

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
fa-legacy-reactjs-org Ignored Ignored Preview Oct 4, 2026 12:50pm UTC

Request Review

@mahdyaralipor mahdyaralipor changed the title Fix label_core_team_prs.yml: upstream renamed facebook/react → react/react Fix workflows: upstream renamed facebook/react → react/react Oct 4, 2026
@mahdyaralipor

mahdyaralipor commented Oct 4, 2026

Copy link
Copy Markdown
Author

Hi maintainers! This PR is ready for review. It fixes the broken Label Core Team PRs and Discord Notify workflows (both fail on every PR with workflow was not found because they still reference facebook/react, which was renamed to react/react). Just a 2-line change — would appreciate a look. Thanks! 🙏

maqsood576
maqsood576 approved these changes Nov 1, 2026
View reviewed changes

@maqsood576 maqsood576 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The check_maintainer job defines the uses key twice with different reusable workflow references. YAML does not support duplicate keys reliably, and a GitHub Actions job can reference only one reusable workflow. Please verify the intended repository and keep a single uses entry.

@maqsood576

maqsood576 commented Nov 1, 2026

Copy link
Copy Markdown

The check_maintainer job defines the uses key twice with different reusable workflow references. YAML does not support duplicate keys reliably, and a GitHub Actions job can reference only one reusable workflow. Please verify the intended repository and keep a single uses entry.

This branch was successfully deployed

1 active deployment
Preview – fa-react-dev — 97ea57c9 Deployed Oct 4, 2026 by vercel[bot]
View deployment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@maqsood576 maqsood576 maqsood576 approved these changes
Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mahdyaralipor @maqsood576