Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Auto publication] Force job serialization #209

Merged
merged 1 commit into from
Apr 30, 2021
Merged

Conversation

tidoust
Copy link
Member

@tidoust tidoust commented Apr 30, 2021

One job runs for each of the 3 documents in the repository. Each of them updates the "gh-pages" branch. Problem is that these jobs are typically run in parallel, leading to "not up to date" issues when "git push" commands are run.

For an example of such a problem, see:
https://github.com/w3c/webcodecs/runs/2462176606

Also see discussion in:
w3c/spec-prod#58

This update tells GitHub to serialize the jobs. Since the jobs are relatively independent otherwise, it also tells GitHub to run them to completion even when one of them fails.

One job runs for each of the 3 documents in the repository. Each of them
updates the "gh-pages" branch. Problem is that these jobs are typically run in
parallel, leading to "not up to date" issues when "git push" commands are run.

For an example of such a problem, see:
https://github.com/w3c/webcodecs/runs/2462176606

Also see discussion in:
w3c/spec-prod#58

This update tells GitHub to serialize the jobs. Since the jobs are relatively
independent otherwise, it also tells GitHub to run them to completion even
when one of them fails.
@tidoust tidoust requested a review from chcunningham April 30, 2021 08:24
@chcunningham
Copy link
Collaborator

Thanks @tidoust

@chcunningham chcunningham merged commit 8b69591 into main Apr 30, 2021
github-actions bot added a commit that referenced this pull request Apr 30, 2021
SHA: 8b69591
Reason: push, by @chcunningham

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Apr 30, 2021
SHA: 8b69591
Reason: push, by @chcunningham

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Apr 30, 2021
SHA: 8b69591
Reason: push, by @chcunningham

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@chcunningham chcunningham deleted the max-parallel branch June 2, 2021 03:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants