Skip to content

[code-infra] Prefer upstream/origin when inferring publish remote#1427

Merged
Janpot merged 1 commit into
masterfrom
fix-publish-remote-priority
Jun 11, 2026
Merged

[code-infra] Prefer upstream/origin when inferring publish remote#1427
Janpot merged 1 commit into
masterfrom
fix-publish-remote-priority

Conversation

@Janpot

@Janpot Janpot commented May 7, 2026

Copy link
Copy Markdown
Member

getRepositoryInfo() previously returned the first mui/* GitHub remote it encountered in git remote -v output, which is order-dependent and can pick the wrong remote when multiple mui/* remotes are configured (e.g. a mui/material-ui clone that also has a mui/material-ui-docs remote, or a developer fork listed before the canonical remote).

Now we collect all mui/* remotes and select by priority: upstreamorigin → first one seen.

@Janpot Janpot requested a review from a team May 7, 2026 09:27
@Janpot Janpot added the scope: code-infra Involves the code-infra product (https://www.notion.so/mui-org/5562c14178aa42af97bc1fa5114000cd). label May 7, 2026
@Janpot Janpot marked this pull request as ready for review May 7, 2026 10:30
@Janpot Janpot requested a review from brijeshb42 May 11, 2026 15:05
@github-actions

Copy link
Copy Markdown

This pull request has been inactive for 30 days. Please remove the stale label or leave a comment to keep it open. Otherwise, it will be closed in 15 days.

@github-actions github-actions Bot added the stale Inactive for 7 days (issues) or 30 days (PRs); closed after 5 or 15 more days if no update. label Jun 11, 2026
@Janpot Janpot merged commit 569d8c8 into master Jun 11, 2026
14 of 15 checks passed
@Janpot Janpot deleted the fix-publish-remote-priority branch June 11, 2026 04:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: code-infra Involves the code-infra product (https://www.notion.so/mui-org/5562c14178aa42af97bc1fa5114000cd). stale Inactive for 7 days (issues) or 30 days (PRs); closed after 5 or 15 more days if no update.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants