-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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
bugfix: replace flatten-maven-plugin
with easyj-maven-plugin
to fix the conflict between shade
and flatten
#4626
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #4626 +/- ##
=============================================
+ Coverage 48.90% 49.04% +0.13%
- Complexity 4042 4056 +14
=============================================
Files 734 734
Lines 25577 25577
Branches 3156 3156
=============================================
+ Hits 12509 12544 +35
+ Misses 11732 11698 -34
+ Partials 1336 1335 -1
|
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.
LGTM 要不要顺便把版本号改为1.5.2-SNAPSHOT ?
done |
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.
LGTM
等下,插件有个优化,我刚提交,等上传OSS后,我升下版本。 |
shade
and flatten
, replace flatten-maven-plugin
with easyj-maven-plugin
shade
and flatten
, replace flatten-maven-plugin
with easyj-maven-plugin
flatten-maven-plugin
with easyj-maven-plugin
to fix the conflict between shade
and flatten
|
…ix the conflict between `shade` and `flatten` (apache#4626)
bugfix: replace
flatten-maven-plugin
witheasyj-maven-plugin
to fix the conflict betweenshade
andflatten
问题修复:使用
easyj-maven-plugin
插件代替flatten-maven-plugin
插件,以修复shade
插件与flatten
插件不兼容的问题,导致seata-all.pom
中的${revision}
并没有被替换为具体的版本号,导致应用端引用后打包失败。另外,
easyj-maven-plugin
插件还可以简化所有pom文件。fixes #4618
easyj-maven-plugin
插件源码:https://github.com/easyj-projects/easyj-maven-plugin/tree/develop