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

👷 Update LCOV to Version 2.2 #160

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

👷 Update LCOV to Version 2.2 #160

wants to merge 11 commits into from

Conversation

pcaversaccio
Copy link
Owner

@pcaversaccio pcaversaccio commented Dec 17, 2024

🕓 Changelog

This PR updates the LCOV version to the latest release, 2.2, in the coverage CI pipeline. It also introduces a temporary workaround for the Verify minimum coverage step, as LCOV 2.2 is not yet supported in zgosalvez/github-actions-report-lcov@v4, as outlined in issue #168. Additionally, the romeovs/[email protected] has been updated and is now triggered for pull_request_target events as well (see PR #63).

🐶 Cute Animal Picture

image

Signed-off-by: Pascal Marco Caversaccio <[email protected]>
Signed-off-by: Pascal Marco Caversaccio <[email protected]>
Signed-off-by: Pascal Marco Caversaccio <[email protected]>
Signed-off-by: Pascal Marco Caversaccio <[email protected]>
Signed-off-by: Pascal Marco Caversaccio <[email protected]>
Signed-off-by: Pascal Marco Caversaccio <[email protected]>
Signed-off-by: Pascal Marco Caversaccio <[email protected]>
@pcaversaccio pcaversaccio changed the title 👷 Update lcov to Version 2.2 👷 Update LCOV to Version 2.2 Dec 17, 2024
@pcaversaccio pcaversaccio self-assigned this Dec 17, 2024
Signed-off-by: Pascal Marco Caversaccio <[email protected]>
@pcaversaccio pcaversaccio added this to the 1.0.0 milestone Dec 17, 2024
@pcaversaccio pcaversaccio added ci/cd 👷‍♂️ CI/CD configurations refactor/cleanup ♻️ Code refactorings and cleanups labels Dec 17, 2024
Signed-off-by: Pascal Marco Caversaccio <[email protected]>
Signed-off-by: Pascal Marco Caversaccio <[email protected]>
Signed-off-by: Pascal Marco Caversaccio <[email protected]>
Copy link

`CreateX` Test Coverage Report

Coverage after merging ci/update-lcov into main will be
100.00%
Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
src
   CreateX.sol100%100%100%100%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/cd 👷‍♂️ CI/CD configurations refactor/cleanup ♻️ Code refactorings and cleanups
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant