Skip to content

Meta: use gh-action-mutex in the pr-preview action#3821

Merged
ljharb merged 1 commit intomainfrom
publish-action-mutex
Apr 24, 2026
Merged

Meta: use gh-action-mutex in the pr-preview action#3821
ljharb merged 1 commit intomainfrom
publish-action-mutex

Conversation

@michaelficarra
Copy link
Copy Markdown
Member

This will avoid the problem where the PR preview push to gh-pages fails when multiple PRs are running through CI at the same time. Now they will queue up and run in sequence.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 8, 2026

The rendered spec preview for this PR is available as a single page at https://tc39.es/ecma262/pr/3821 and as multiple pages at https://tc39.es/ecma262/pr/3821/multipage .

@michaelficarra
Copy link
Copy Markdown
Member Author

Alternatively, I guess we could make the job rebase and try to push again? There'd still be a race condition though. If we wanted to take a combined approach, we could just acquire the mutex for that push/rebase loop.

Copy link
Copy Markdown
Member

@bakkot bakkot left a comment

Choose a reason for hiding this comment

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

Assuming this works as described, sure

@michaelficarra
Copy link
Copy Markdown
Member Author

I don't have a great way to test it without a bunch of noise. But we should know pretty quickly whether it works or not after merging.

Copy link
Copy Markdown
Member

@ljharb ljharb left a comment

Choose a reason for hiding this comment

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

looks good

@michaelficarra michaelficarra added the ready to merge Editors believe this PR needs no further reviews, and is ready to land. label Apr 10, 2026
@ljharb ljharb force-pushed the publish-action-mutex branch from 10f526b to 2314d2b Compare April 24, 2026 05:34
@ljharb ljharb merged commit 2314d2b into main Apr 24, 2026
11 of 12 checks passed
@ljharb ljharb deleted the publish-action-mutex branch April 24, 2026 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

meta ready to merge Editors believe this PR needs no further reviews, and is ready to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants