-
Notifications
You must be signed in to change notification settings - Fork 8
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
Execution hangs without any visible error #22
Comments
@cristiammercado Hi, thanks for trying to use my reporter. I tested it using Gradle only and haven't received any issues so far. I will test using Maven and see whats causing this. Ideally error should not come but if it comes it should be valid and meaningful one. I will get back to you in a day! |
@cristiammercado I just forked one of the Java Maven Repo from opensource and my reporter worked right out of the box with out errors. https://github.com/PavanMudigonda/java-maven-playground/actions/runs/2698921071 I am not sure whats causing your repo to hang. Could you tell me if the JaCoCo Coverage XML was generated and if location was correct ? usually this is where issues happen. |
I am using the Maven plugin dependency Thanks in advance! |
Ok. This info is helpful. Let me get back! |
@cristiammercado Could you let me know if you got this error ? I got this error with the jacoco.xml file you provided yesterday. |
@cristiammercado please try again with v4.8 |
I get exactly the same error as you posted above using |
It seems this could be case when you have no missing lines and 100% coverage is achieved. I will try to fix it this week ! |
Even if I'd love this to be the reason, I can assure you that this is not the case for my codebase ;-) For some packages, we have 100% line coverage and no missed lines, but not overall. |
would you be able to give me a screenshot of your error ? is it exact same issue from above. |
Sure thing. This is my step configuration:
I added a debug step before the reporter, to show that the jacoco.xml is there. |
@benediktarnold if is possible for you to put shell script “cat jacoco.xml” and check if its empty or code coverage is present in that xml ? If you see below screenshot, this is how it’s supposed to work. it should convert the jacoco xml, to markdown file, and publish report. I suspect if your xml is empty. I just need this to be confirmed. if xml is valid then I am doing something wrong. I use jacoco-maven-plugin 0.8.8 |
@benediktarnold @cristiammercado It seems issue is happening when there are 61,000 plus lines in your Coverage Report ! |
Describe the bug
I have added this action today to see JaCoCo's coverage reports. Right now I'm using it for
feature/**
project branches, but the execution of the entire GitHub workflow hangs, especially in the part shown in the screenshot below.The configuration of my action is very basic, it is a Maven project, I share it below:
To Reproduce
Steps to reproduce the behavior:
Expected behavior
It is expected to run normally, or if there is an error it will show it and end the workflow.
Screenshots
Desktop (please complete the following information):
Not apply
Smartphone (please complete the following information):
Not apply
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: