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

ci: ignore failures on Ninja summary #26914

Merged
merged 1 commit into from
Dec 10, 2020
Merged

ci: ignore failures on Ninja summary #26914

merged 1 commit into from
Dec 10, 2020

Conversation

jkleinsc
Copy link
Contributor

@jkleinsc jkleinsc commented Dec 9, 2020

Description of Change

Occasionally CircleCI builds fail while printing the ninja summary of the build. Since this report is informational only, this PR changes that step so that any failures are ignored.

Here's an example of a failed build: https://app.circleci.com/pipelines/github/electron/electron/33723/workflows/138db88d-eacb-4421-8ea9-3b1a6c5cef81/jobs/741116

Checklist

Release Notes

Notes: none

@jkleinsc jkleinsc added semver/patch backwards-compatible bug fixes and removed semver/patch backwards-compatible bug fixes labels Dec 10, 2020
@jkleinsc jkleinsc merged commit f876154 into master Dec 10, 2020
@release-clerk
Copy link

release-clerk bot commented Dec 10, 2020

No Release Notes

@jkleinsc jkleinsc deleted the ignore-stats-errors branch December 10, 2020 00:58
@trop
Copy link
Contributor

trop bot commented Dec 14, 2020

The backport process for this PR has been manually initiated, here we go! :D

@trop
Copy link
Contributor

trop bot commented Dec 14, 2020

I have automatically backported this PR to "10-x-y", please check out #26993

@trop
Copy link
Contributor

trop bot commented Dec 14, 2020

I have automatically backported this PR to "12-x-y", please check out #26994

@trop
Copy link
Contributor

trop bot commented Dec 14, 2020

I have automatically backported this PR to "9-x-y", please check out #26995

@trop
Copy link
Contributor

trop bot commented Dec 14, 2020

I have automatically backported this PR to "11-x-y", please check out #26996

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants