Skip to content

[internal] Fix use of ellipsis#1245

Open
oliviertassinari wants to merge 2 commits intomasterfrom
fix-ellispse
Open

[internal] Fix use of ellipsis#1245
oliviertassinari wants to merge 2 commits intomasterfrom
fix-ellispse

Conversation

@oliviertassinari
Copy link
Copy Markdown
Member

@oliviertassinari oliviertassinari commented Mar 25, 2026

@oliviertassinari oliviertassinari added internal Behind-the-scenes enhancement. Formerly called “core”. type: bug It doesn't behave as expected. labels Mar 25, 2026
@oliviertassinari oliviertassinari temporarily deployed to fix-ellispse - mui-tools-public PR #1245 March 25, 2026 13:25 — with Render Destroyed
@Janpot
Copy link
Copy Markdown
Member

Janpot commented Mar 25, 2026

I'm using this to refine https://github.com/oliviertassinari/mui-olivier-tools/blob/master/scripts/propagate-create-prs.sh

btw, I've been using claude code for creating and maintaining prs in bullk across repos. Works quite well.

You need to rebase, recent work introduced authentication to the ci report upload logic

@oliviertassinari
Copy link
Copy Markdown
Member Author

oliviertassinari commented Mar 25, 2026

I've been using claude code for creating and maintaining prs in bullk across repos. Works quite well.

Claude is creating those scripts, but I don't trust Claude for ongoing use. I want it to be deterministic so I can blindly trust it. I also want to fix bugs as I find them, to get something super smooth. This kind of smooth https://youtu.be/MiUHjLxm3V0?si=0B1xzz96MKuDfSvO&t=815 😁

Ok, the script works. In this case, I'm creating PRs because I want people to learn how to correctly use (https://vercel.com/design/guidelines covers this in more depth) and I can't see any way to automate the logic. I would have pushed straight to master otherwise 😁

@Janpot
Copy link
Copy Markdown
Member

Janpot commented Mar 25, 2026

I want it to be deterministic

I found that in many cases it can't be deterministic as there are subtle differences between repos.

@mui-bot
Copy link
Copy Markdown

mui-bot commented Mar 25, 2026

Bundle size report

Bundle size will be reported once CircleCI build #27359 finishes.

Status: 🟠 Processing...


Check out the code infra dashboard for more information about this PR.

@oliviertassinari
Copy link
Copy Markdown
Member Author

oliviertassinari commented Mar 25, 2026

I found that in many cases it can't be deterministic as there are subtle differences between repos

Ah, yes, the new script I'm testing here https://github.com/oliviertassinari/mui-olivier-tools/blob/master/scripts/propagate-create-prs.sh, is only automating the creation of PRs from the working tree.

For a blind replication of the git diff itself, I have a different script #1230 (comment). If there are subtle differences between repositories, I hope that this will help find them and make them identical 😁.

@oliviertassinari oliviertassinari temporarily deployed to fix-ellispse - code-infra-dashboard PR #1245 March 25, 2026 14:23 — with Render Destroyed
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 3, 2026

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 May 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Behind-the-scenes enhancement. Formerly called “core”. stale Inactive for 7 days (issues) or 30 days (PRs); closed after 5 or 15 more days if no update. type: bug It doesn't behave as expected.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants