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 adding coverage to PRs #131

Merged
merged 2 commits into from
Oct 17, 2024
Merged

Fix adding coverage to PRs #131

merged 2 commits into from
Oct 17, 2024

Conversation

grzegorz8
Copy link
Member

@grzegorz8 grzegorz8 commented Oct 17, 2024

Description

Step "Add coverage to PR" fails with the following error:

Error: ENOENT: no such file or directory, open '/home/runner/work/flink-http-connector/flink-http-connector/target/site/jacoco/jacoco.xml'

Adding ${argLine} as below solves the issue.

<argLine>${argLine} ...</argLine>

@grzegorz8 grzegorz8 changed the title Update madrapps/jacoco-report to v1.7.1 Fix adding coverage to PRs Oct 17, 2024
@grzegorz8 grzegorz8 force-pushed the bump-jacoco-version branch from 4c6a2b5 to b97ecec Compare October 17, 2024 13:01
@getindata getindata deleted a comment from github-actions bot Oct 17, 2024
Copy link
Contributor

Overall Project 93.61% 🍏

There is no coverage information present for the Files changed

Copy link
Contributor

Overall Project 93.61% 🍏

There is no coverage information present for the Files changed

@grzegorz8 grzegorz8 merged commit 3f86bf4 into main Oct 17, 2024
3 checks passed
@MarekMaj MarekMaj deleted the bump-jacoco-version branch October 17, 2024 14:22
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.

2 participants