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

Use Gradle 8 compatible API #83

Merged
merged 3 commits into from
Apr 10, 2024
Merged

Conversation

shemnon
Copy link
Contributor

@shemnon shemnon commented Apr 8, 2024

What does this PR do?

In Web3JGradle plugin replace an API that was removed in Gradle 8 with a
Gradle 8 API and an appropriate fallback.

Where should the reviewer start?

PR is only a few lines, should be self-evident.

Why is it needed?

The old getOutputDir API has been removed in Gradle 8. This prevents
the plugin from being used in Gradle 8. The chagnes use a compatable API
that also falls back to the original code if used in a very old Gradle 7
build.

In Web3JGradle plugin replace an API that was removed in Gradle 8 with a
Gradle 8 API and an appropriate fallback.

Signed-off-by: Danno Ferrin <danno@numisight.com>
Copy link
Contributor

@gtebrean gtebrean left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please fix the code format in order for build to pass

@@ -2,6 +2,6 @@ group=org.web3j
version=4.11.2
org.gradle.caching=true
org.gradle.parallel=true
solidityPluginVersion=0.4.0
solidityPluginVersion=0.4.1-SNAPSHOT
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

revert this change because we don't provide SNAPSHOT versions for gradle plugins
Solidity plugin version will be updated here before releasing new gradle plugin

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

Signed-off-by: Danno Ferrin <danno@numisight.com>
Signed-off-by: Danno Ferrin <danno@numisight.com>
@NickSneo NickSneo merged commit c98501d into hyperledger:master Apr 10, 2024
2 checks passed
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.

None yet

3 participants