Skip to content

Allow CI to pass even if the processcoverage step fails#901

Merged
giordano merged 2 commits intomasterfrom
dpa/process-coverage-continue-on-error
Apr 29, 2025
Merged

Allow CI to pass even if the processcoverage step fails#901
giordano merged 2 commits intomasterfrom
dpa/process-coverage-continue-on-error

Conversation

@DilumAluthge
Copy link
Member

No description provided.

- uses: julia-actions/julia-processcoverage@v1
# Allow CI to pass even if the `processcoverage` step fails:
continue-on-error: true
- uses: codecov/codecov-action@v5
Copy link
Member

Choose a reason for hiding this comment

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

Also here

Copy link
Member Author

Choose a reason for hiding this comment

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

IIRC, the default behavior for codecov-action is to not fail CI on a Codecov error: https://github.com/codecov/codecov-action/blob/ad3126e916f78f00edff4ed0317cf185271ccc2d/action.yml#L48-L51

So I don't think we need continue-on-error for the codecov-action step.

Copy link
Member

Choose a reason for hiding this comment

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

I'd prefer not to rely on defaults, but whatever.

Copy link
Member Author

Choose a reason for hiding this comment

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

@DilumAluthge DilumAluthge marked this pull request as ready for review April 29, 2025 07:29
@DilumAluthge DilumAluthge requested a review from giordano April 29, 2025 07:30
@DilumAluthge DilumAluthge force-pushed the dpa/process-coverage-continue-on-error branch from abc61a4 to 222673b Compare April 29, 2025 08:06
@giordano giordano merged commit 042ddc2 into master Apr 29, 2025
40 of 49 checks passed
@giordano giordano deleted the dpa/process-coverage-continue-on-error branch April 29, 2025 08:29
@giordano giordano added the CI label Apr 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants