Skip to content

Conversation

@alaneuler
Copy link
Contributor

fix: #699

@sofastack-bot sofastack-bot bot added bug Something isn't working cla:yes size/S labels Sep 7, 2020
@codecov-commenter
Copy link

codecov-commenter commented Sep 7, 2020

Codecov Report

Merging #700 into master will decrease coverage by 0.01%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #700      +/-   ##
============================================
- Coverage     52.05%   52.04%   -0.02%     
  Complexity       78       78              
============================================
  Files           295      295              
  Lines          8302     8304       +2     
  Branches       1155     1156       +1     
============================================
  Hits           4322     4322              
- Misses         3526     3527       +1     
- Partials        454      455       +1     
Impacted Files Coverage Δ Complexity Δ
...a/runtime/invoke/DynamicJvmServiceProxyFinder.java 50.00% <0.00%> (-0.73%) 0.00 <0.00> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1808e7d...6971d61. Read the comment docs.

Copy link
Member

@elseifer elseifer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

test pass

@alaneuler alaneuler merged commit f58f2ef into master Sep 8, 2020
@alaneuler alaneuler deleted the fix/ark.jvm.invoke branch October 26, 2020 03:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working cla:yes size/S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

When using sofa:reference to reference jvm service in ark environment, my app occurred an NPE error

4 participants