-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
[issues#12551] nacos-client modularity problem #12696
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ 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.
|
client/pom.xml
Outdated
@@ -22,24 +22,24 @@ | |||
<version>${revision}</version> | |||
<relativePath>../pom.xml</relativePath> | |||
</parent> | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
请不要修改缩进, 使用nacos-code-style进行reformat
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
已修复,ide忽略了空格差异显示,没感知到
#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