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

Continue CI on error #2081

Merged
merged 4 commits into from
May 2, 2022
Merged

Continue CI on error #2081

merged 4 commits into from
May 2, 2022

Conversation

rikhuijzer
Copy link
Collaborator

@rikhuijzer rikhuijzer commented Apr 30, 2022

Currently, all CI test jobs get cancelled as soon as one job fails. This is very useful to save GitHub Actions minutes. However, Pluto's tests are a bit flakely which often causes all jobs to be cancelled for no good reason. Even worse, in many cases, this happens after 20 minutes. So, when the jobs are almost done anyway! Instead of getting a lot of valuable information, we now get just a bunch of "cancelled" reports.

EDIT: Got it. Other CI configs set fail-fast: false explicitly. For example, https://github.com/JuliaData/DataFrames.jl/blob/main/.github/workflows/ci.yml.

@rikhuijzer rikhuijzer requested a review from fonsp April 30, 2022 11:29
@github-actions
Copy link
Contributor

Try this Pull Request!

Open Julia and type:

julia> import Pkg
julia> Pkg.activate(temp=true)
julia> Pkg.add(url="https://github.com/fonsp/Pluto.jl", rev="rh/continue-on-error")
julia> using Pluto

@rikhuijzer rikhuijzer added the tests issues that improve the code coverage and the automatic testing label Apr 30, 2022
@rikhuijzer rikhuijzer marked this pull request as draft April 30, 2022 11:46
@rikhuijzer rikhuijzer marked this pull request as ready for review April 30, 2022 11:48
@rikhuijzer rikhuijzer merged commit a306cb7 into main May 2, 2022
@rikhuijzer rikhuijzer deleted the rh/continue-on-error branch May 2, 2022 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests issues that improve the code coverage and the automatic testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants