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
Remove custom Surefire provider #1598
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1598 +/- ##
============================================
- Coverage 92.07% 91.94% -0.14%
+ Complexity 3561 3491 -70
============================================
Files 325 321 -4
Lines 8510 8329 -181
Branches 742 722 -20
============================================
- Hits 7836 7658 -178
+ Misses 502 500 -2
+ Partials 172 171 -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.
Please also remove it from overview.adoc
in the the user guide and add an entry to the release notes.
LICENSE.md
Outdated
@@ -3,7 +3,6 @@ Open Source Licenses | |||
|
|||
The individual JUnit modules/artifacts use different open source licenses: | |||
|
|||
- `junit-platform-surefire-provider` uses [Apache License v2.0](junit-platform-surefire-provider/LICENSE.md) | |||
- All other modules use [Eclipse Public License v2.0](junit-jupiter-api/LICENSE.md). | |||
- All modules use [Eclipse Public License v2.0](junit-jupiter-api/LICENSE.md). |
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.
I think we should just replace this file with the EPL v2 now.
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.
✔️
@@ -209,7 +209,7 @@ additional dependency to the runtime classpath. | |||
==== Maven | |||
|
|||
NOTE: The custom `junit-platform-surefire-provider`, which was originally developed by | |||
the JUnit team, has been deprecated and is scheduled to be removed in JUnit Platform 1.4. | |||
the JUnit team, is removed from JUnit Platform 1.4. |
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.
was removed?
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.
✔️
532e83f
to
3b75241
Compare
@marcphilipp Anything else to erase? |
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.
Actually, yes. We can remove the LICENSE.md
files from all subprojects now.
build.gradle
Outdated
: 'EPL-2.0' | ||
] | ||
} | ||
licenseOf = { project -> licenses['EPL-2.0'] } |
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.
we only need license
now
license = [
name: 'Eclipse Public License v2.0',
url: 'http://www.eclipse.org/legal/epl-v20.html',
headerFile: 'eclipse-public-license-2.0.java'
]
Only EPL-2.0 is used, no need to lookup a license per project. Issue #1597
This is it. |
Overview
Closes #1597
I hereby agree to the terms of the JUnit Contributor License Agreement.
Definition of Done
@API
annotations