-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Deprecate alternative JVM plugins #11598
Deprecate alternative JVM plugins #11598
Conversation
@@ -31,7 +30,6 @@ | |||
/** | |||
* Options for Scala platform compilation. | |||
*/ | |||
@Incubating |
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.
This type is also used by the "normal" scala plugin and is thus not deprecated. It should not be incubating.
...rojects/language-java/src/main/java/org/gradle/language/java/plugins/JavaLanguagePlugin.java
Outdated
Show resolved
Hide resolved
...testFixtures/groovy/org/gradle/api/tasks/compile/AbstractCachedCompileIntegrationTest.groovy
Outdated
Show resolved
Hide resolved
I think we should document this in some form. We will just need to be careful about the wording. |
1aa9473
to
435cc83
Compare
Thanks for the feedback @adammurdoch . I added a section to the upgrading guide: 9ebe8e4 Before I adjust all tests, I would like to decide on a deprecation warning. Proposal (would add the same message for all deprecations):
Renders to:
|
5943ad5
to
fad870e
Compare
fad870e
to
17485f5
Compare
This was deprecated as part of #11598. We missed that it can also be used in artifact resolution queries which are not (yet) deprecated. For example: dependencies { createArtifactResolutionQuery().withArtifacts(JvmLibrary, ...) }
This was deprecated as part of #11598. We missed that it can also be used in artifact resolution queries which are not (yet) deprecated. For example: dependencies { createArtifactResolutionQuery().withArtifacts(JvmLibrary, ...) }
See #9837
Notes:
@Deprecated
to all API types that are exclusively used by the deprecated plugins. In particular, I checked everything that was still incubating in the corresponding projects, because a type that has been around since Gradle 2.x/3.x should either be deprecated or be de-incubated. There is one type,JavadocArtifact
, which I left incubating but did not deprecate (yet) because it is also used in the artifact query API (which we will probably also deprecate soonish).