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

Fix "Update Editor's Copy" workflow #714

Merged
merged 1 commit into from
Mar 3, 2022
Merged

Fix "Update Editor's Copy" workflow #714

merged 1 commit into from
Mar 3, 2022

Conversation

irori
Copy link
Collaborator

@irori irori commented Mar 3, 2022

Fixes #704.

It was failing because bikeshed was not installed in the martinthomson/i-d-template@v1 docker image. Installing bikeshed in it is not trivial (that needs to build native extensions but the docker image does not have gcc), so this patch lets bikeshed run outside the docker environment.

This also makes this action publish subresource-loading.html.

It was failing because bikeshed was not installed in the
`martinthomson/i-d-template@v1` docker image. Installing bikeshed in it
is not trivial (that needs to build native extensions but the docker
image does not have gcc), so this patch lets bikeshed run outside the
docker environment.

This also makes this action publish `subresource-loading.html`.
@irori irori requested a review from hayatoito March 3, 2022 02:43
@hayatoito
Copy link
Collaborator

Looks great! Thanks!
I thought this can be solved in upstream, but it is totally fine to have a local fix.

@hayatoito hayatoito merged commit 87e5b4a into WICG:main Mar 3, 2022
github-actions bot added a commit that referenced this pull request Mar 3, 2022
SHA: 87e5b4a
Reason: push, by @hayatoito

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 Mar 3, 2022
SHA: 87e5b4a
Reason: push, by @hayatoito

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@irori irori deleted the bikeshed branch March 3, 2022 04:30
github-actions bot added a commit to hayatoito/webpackage that referenced this pull request Mar 7, 2022
SHA: 87e5b4a
Reason: push, by @hayatoito

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit to hayatoito/webpackage that referenced this pull request Mar 7, 2022
SHA: 87e5b4a
Reason: push, by @hayatoito

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 1, 2022
SHA: 87e5b4a
Reason: push, by @hayatoito

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit to hayatoito/webpackage that referenced this pull request Apr 22, 2022
SHA: 87e5b4a
Reason: push, by @hayatoito

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

GitHub Action: "Update Editor's Copy" is failing.
2 participants