Skip to content

Comments

Update README.md Gradle and Java version instructions#654

Merged
klp2 merged 5 commits intomainfrom
greg/eng-3756-update-gradle-instructions-in-readmemd-files
Jan 8, 2026
Merged

Update README.md Gradle and Java version instructions#654
klp2 merged 5 commits intomainfrom
greg/eng-3756-update-gradle-instructions-in-readmemd-files

Conversation

@oschwald
Copy link
Member

@oschwald oschwald commented Jan 8, 2026

Summary

  • Update Gradle dependency configuration from deprecated compile to implementation
  • Update Java version requirement from Java 11 to Java 17 to match actual requirements

Resolves ENG-3756

Test plan

  • Verify Gradle instructions work with modern Gradle versions
  • Confirm Java 17 requirement matches pom.xml <release>17</release>

🤖 Generated with Claude Code

oschwald and others added 3 commits January 8, 2026 13:20
Replace deprecated `compile` configuration with `implementation` in
the Gradle instructions. The `compile` configuration was deprecated in
Gradle 3.4 and removed in Gradle 7.0.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The library requires Java 17+ as it uses Java records and the pom.xml
specifies <release>17</release>.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Replace deprecated getXXX() style methods with the modern Java record
accessor methods throughout all code examples. The deprecated methods
are scheduled for removal in version 6.0.0.

Changes include:
- response.getCountry() -> response.country()
- country.getIsoCode() -> country.isoCode()
- country.getName() -> country.name()
- response.getMostSpecificSubdivision() -> response.mostSpecificSubdivision()
- location.getLatitude() -> location.latitude()
- And similar changes throughout all examples

Also fixed incorrect return type in Anonymous Plus example
(AnonymousIpResponse -> AnonymousPlusResponse).

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@oschwald oschwald force-pushed the greg/eng-3756-update-gradle-instructions-in-readmemd-files branch from c93f286 to 385f239 Compare January 8, 2026 21:43
Update the City, Connection-Type, Domain, and ISP database examples
to use try-with-resources for proper resource management, matching
the pattern already used in the Anonymous IP, Anonymous Plus, ASN,
and Enterprise examples.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@klp2
Copy link
Contributor

klp2 commented Jan 8, 2026

It's not quite part of this PR, but should

<target>11</target>
also be 17?

@oschwald
Copy link
Member Author

oschwald commented Jan 8, 2026

I think we can just drop source and target entirely now. release supersedes it. I'll push that change.

@klp2 klp2 merged commit 2cc0f3e into main Jan 8, 2026
24 checks passed
@klp2 klp2 deleted the greg/eng-3756-update-gradle-instructions-in-readmemd-files branch January 8, 2026 22:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants