Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

fix:fix serializeTransform function to handle class[] ,and return org… #486

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

linwaiwai
Copy link

[fix serializeTransform function to handle class[] ,and return orginal when the target loader and sourceLoader are the same](fix:fix serializeTransform function to handle class[] ,and return orginal when the target loader and sourceLoader are the same)

Motivation

Try to fix the function call throught bizs normally

Modification

fix serializeTransform function

Result

Resolved or fixed #koupleless/koupleless#81.

If there is no issue then describe the changes introduced by this PR.

…inal when the target loader and sourceLoader are the same
Copy link

sofastack-cla bot commented Feb 6, 2024

Hi @linwaiwai, welcome to SOFAStack community, Please sign Contributor License Agreement!

After you signed CLA, we will automatically sync the status of this pull request in 3 minutes.

Copy link

codecov bot commented Feb 7, 2024

Codecov Report

Attention: 13 lines in your changes are missing coverage. Please review.

Comparison is base (17a685e) 80.98% compared to head (34f2230) 79.12%.
Report is 112 commits behind head on master.

Files Patch % Lines
...ay/sofa/serverless/common/util/SerializeUtils.java 31.57% 9 Missing and 4 partials ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #486      +/-   ##
============================================
- Coverage     80.98%   79.12%   -1.86%     
+ Complexity       66       63       -3     
============================================
  Files            45       47       +2     
  Lines          1031     1241     +210     
  Branches         90      118      +28     
============================================
+ Hits            835      982     +147     
- Misses          145      195      +50     
- Partials         51       64      +13     

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant