Skip to content
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

Fix #1413: Replace deprecated JavaPluginConventions usage #1416

Merged
merged 5 commits into from
Jun 26, 2023

Conversation

TWiStErRob
Copy link
Contributor

@TWiStErRob TWiStErRob commented Jun 24, 2023

Pull Request Details

Description

Replaces .convention usage with .extension and JavaPluginConvention with JavaPluginExtension.
JavaPluginExtension was added in Gradle 4.10. Consdering the docs say:

This project requires Gradle 7.3 or newer. -- docs

this change should bear very low risk.

Related Issue

Fixes #1413

Motivation and Context

Gradle is removing long-deprecated conventions APIs in favor of extensions. In Gradle 8.2 (now in RC2) they added nagging so plugin users will be nagged but they can't action the nagging until there's a version of the plugin released which doesn't use conventions.

How Has This Been Tested

gradlew build
and
gradlew :publishSnapshotPublicationToMavenLocal + mavenLocal() repo, because includeBuild approach doesn't work as described in CONTRIBUTING.md.

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have included my change in the CHANGELOG.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

hsz and others added 5 commits June 21, 2023 09:59
Bumps [JetBrains/qodana-action](https://github.com/JetBrains/qodana-action) from 2023.1.4 to 2023.1.5.
- [Release notes](https://github.com/JetBrains/qodana-action/releases)
- [Commits](JetBrains/qodana-action@v2023.1.4...v2023.1.5)

---
updated-dependencies:
- dependency-name: JetBrains/qodana-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
@hsz hsz merged commit 483a62c into JetBrains:master Jun 26, 2023
@TWiStErRob TWiStErRob deleted the fix1413 branch June 26, 2023 09:26
@hsz
Copy link
Member

hsz commented Jun 26, 2023

Thanks for the help on that, Róbert!

@hsz hsz added this to the 1.14.2 milestone Jun 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Gradle 8.2 compatibility
2 participants