Skip to content

Conversation

@e5LA
Copy link

@e5LA e5LA commented May 3, 2025

Closes #251

@iluwatar iluwatar force-pushed the 251-execution-measurement branch from 1407494 to 5685e13 Compare May 22, 2025 04:48
@iluwatar
Copy link
Owner

Updated CI and rebased this branch.

@iluwatar
Copy link
Owner

Checkstyle complains about code formatting. Please check it out.

Not sure though why Sonar is not detecting the tests...

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@e5LA
Copy link
Author

e5LA commented May 22, 2025

Thanks for checking. I’ve fixed the formatting issues, and ./gradlew check passes locally

Regarding SonarQube, seems that all the tests in the project are not properly detected.

Lines to Cover: 767
Covered Lines: 0

I think the jacoco plugin and reports are missing

@e5LA
Copy link
Author

e5LA commented May 22, 2025

Regarding test coverage in Sonar - I created #260 - please have a look

omarhaweel added a commit to omarhaweel/30-seconds-of-java that referenced this pull request Jun 17, 2025
@stale
Copy link

stale bot commented Jun 21, 2025

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the status: stale label Jun 21, 2025
@e5LA
Copy link
Author

e5LA commented Jun 23, 2025

It's under review.

@stale stale bot removed the status: stale label Jun 23, 2025
@stale
Copy link

stale bot commented Jul 25, 2025

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the status: stale label Jul 25, 2025
@stale
Copy link

stale bot commented Sep 8, 2025

Closed due to inactivity. Thank you for your contributions.

@stale stale bot closed this Sep 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Execution Time Measurement Snippet

2 participants