-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
Provide an all-in-one package for easier usage #1023
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #1023 +/- ##
=============================================
- Coverage 38.27% 38.25% -0.03%
Complexity 1046 1046
=============================================
Files 218 218
Lines 8693 8698 +5
Branches 1083 1085 +2
=============================================
Hits 3327 3327
- Misses 4946 4950 +4
- Partials 420 421 +1
Continue to review full report at Codecov.
|
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.
Thanks.i left some comments.
@@ -0,0 +1,336 @@ | |||
<?xml version="1.0" encoding="UTF-8"?> |
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.
copyright
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.
License header is reqired
@@ -0,0 +1,612 @@ | |||
<?xml version="1.0" encoding="UTF-8"?> |
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.
License header is required
@@ -0,0 +1,336 @@ | |||
<?xml version="1.0" encoding="UTF-8"?> |
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.
License header is required
pom.xml
Outdated
<dependency> | ||
<groupId>org.junit.platform</groupId> | ||
<artifactId>junit-platform-launcher</artifactId> | ||
<version>1.4.2</version> |
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.
Please extract the version number
@xingfudeshi @lovepoem done. |
lgtm |
@zhangthen @ujjboy how to ensure the version consistency of the dependencies in bom.xml and all/pom.xml? |
We can then discuss here #1031 |
Ⅰ. Describe what this PR did
Merge all the following jar‘s code into
seata-all
jar, so that users can get all seata's code by dependent onseata-all
only.seata-bom
jarMove all the third-party jars to
seata-bom
to facilitate the management of third-party jars.Ⅱ. Does this pull request fix one issue?
#914
I had tested the
seata-all
in seata-samples, which will release after this PR release.