Skip to content

Conversation

luxiao0000
Copy link
Contributor

@luxiao0000 luxiao0000 commented Sep 26, 2024

#12551

修正了nacos-client模块化问题

Fixed the nacos-client modularity issue

maven-shade-plugin插件打包时会把gson里的模块化描述文件带过来,导致nacos-client错误的使用了gson的模块化描述文件

When the Maven-shad-plugin plugin was packaged, it brought gson's modular description file to nacos-client, causing NacOS-Client to use gson's modular description file incorrectly

@codecov-commenter
Copy link

codecov-commenter commented Sep 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.60%. Comparing base (077750b) to head (89b40d1).
Report is 3 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##             develop   #12696      +/-   ##
=============================================
- Coverage      69.62%   69.60%   -0.02%     
+ Complexity      9415     9412       -3     
=============================================
  Files           1275     1275              
  Lines          41233    41233              
  Branches        4374     4374              
=============================================
- Hits           28707    28702       -5     
- Misses         10447    10451       +4     
- Partials        2079     2080       +1     

see 6 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

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

client/pom.xml Outdated
@@ -22,24 +22,24 @@
<version>${revision}</version>
<relativePath>../pom.xml</relativePath>
</parent>

Copy link
Collaborator

Choose a reason for hiding this comment

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

请不要修改缩进, 使用nacos-code-style进行reformat

Copy link
Contributor Author

Choose a reason for hiding this comment

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

已修复,ide忽略了空格差异显示,没感知到

@KomachiSion KomachiSion merged commit 85f89bc into alibaba:develop Oct 9, 2024
7 checks passed
@KomachiSion KomachiSion added the area/Client Related to Nacos Client SDK label Oct 9, 2024
@KomachiSion KomachiSion modified the milestones: 2.5.0, 2.4.3 Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/Client Related to Nacos Client SDK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants