Skip to content

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

Open
Janpot wants to merge 1 commit into
masterfrom
fix-publish-remote-priority
Open

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

Conversation

@Janpot
Copy link
Copy Markdown
Member

@Janpot Janpot commented May 7, 2026

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
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).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants