Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[#noissue] Fix missing LauncherSession.close #11888

Merged
merged 1 commit into from
Jan 2, 2025

Conversation

emeroad
Copy link
Member

@emeroad emeroad commented Jan 2, 2025

No description provided.

@emeroad emeroad added the cleanup label Jan 2, 2025
@emeroad emeroad added this to the 3.1.0 milestone Jan 2, 2025
Copy link

codecov bot commented Jan 2, 2025

Codecov Report

Attention: Patch coverage is 0% with 64 lines in your changes missing coverage. Please review.

Project coverage is 37.38%. Comparing base (a3eaff3) to head (2755dfb).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
...unit5/launcher/SharedPluginForkedTestLauncher.java 0.00% 50 Missing ⚠️
...uncher/SharedPluginForkedTestLauncherListener.java 0.00% 4 Missing ⚠️
.../navercorp/pinpoint/test/plugin/util/URLUtils.java 0.00% 3 Missing ⚠️
...inpoint/test/plugin/DefaultPluginTestInstance.java 0.00% 2 Missing ⚠️
...ncher/SharedPluginForkedTestExecutionListener.java 0.00% 2 Missing ⚠️
...inpoint/test/plugin/shared/SharedTestExecutor.java 0.00% 2 Missing ⚠️
...pinpoint/test/plugin/shared/TestThreadFactory.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #11888      +/-   ##
============================================
- Coverage     37.38%   37.38%   -0.01%     
- Complexity    12652    12655       +3     
============================================
  Files          4149     4149              
  Lines        100720   100727       +7     
  Branches      10482    10483       +1     
============================================
+ Hits          37651    37653       +2     
- Misses        60029    60034       +5     
  Partials       3040     3040              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@emeroad emeroad changed the title [#noissue] [#noissue] Fix missing LauncherSession.close [#noissue] Fix missing LauncherSession.close Jan 2, 2025
@emeroad emeroad merged commit eeb6751 into pinpoint-apm:master Jan 2, 2025
1 of 3 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant