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

Add Child Application ID Retrieval Functionality #6181

Closed
wants to merge 1 commit into from

Conversation

Copy link

sonarqubecloud bot commented Dec 5, 2024

Copy link

codecov bot commented Dec 5, 2024

Codecov Report

Attention: Patch coverage is 83.33333% with 3 lines in your changes missing coverage. Please review.

Project coverage is 41.04%. Comparing base (37745ca) to head (343ef66).

Files with missing lines Patch % Lines
...lication/mgt/ApplicationManagementServiceImpl.java 88.23% 1 Missing and 1 partial ⚠️
.../application/mgt/ApplicationManagementService.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #6181      +/-   ##
============================================
- Coverage     45.32%   41.04%   -4.28%     
- Complexity    13830    15406    +1576     
============================================
  Files          1614     1796     +182     
  Lines         99700   123017   +23317     
  Branches      16833    21406    +4573     
============================================
+ Hits          45188    50498    +5310     
- Misses        47868    65034   +17166     
- Partials       6644     7485     +841     
Flag Coverage Δ
unit 27.69% <83.33%> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@dhaura
Copy link
Contributor Author

dhaura commented Dec 9, 2024

Closing this PR since this application service method is no longer needed.

@dhaura dhaura closed this Dec 9, 2024
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.

1 participant