Skip to content

Conversation

@vorburger
Copy link
Collaborator

Re. #242.

Copilot AI review requested due to automatic review settings January 20, 2026 18:06
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR clarifies the supported target Java version in the README documentation, addressing issue #242. The changes improve documentation clarity and formatting throughout the README.

Changes:

  • Added explicit Java 11 runtime requirement documentation
  • Improved formatting with proper code block syntax highlighting and spacing
  • Fixed capitalization of "HTTP API" for consistency
  • Reorganized the "Running tests" section for better clarity

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

README.md Outdated
* [multihash](https://github.com/multiformats/java-multihash)
* [cid](https://github.com/ipld/java-cid)

This library requires (only) Java 11 bytecode & JDK at runtime. (At build time we use current latest LTS tools.)
Copy link

Copilot AI Jan 20, 2026

Choose a reason for hiding this comment

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

The word "(only)" in parentheses seems out of place and unclear. It's not evident what aspect is being emphasized as "only" - is it "only Java 11" (not higher), "only at runtime" (not build time), or something else? Consider removing this word or clarifying the intended meaning.

Suggested change
This library requires (only) Java 11 bytecode & JDK at runtime. (At build time we use current latest LTS tools.)
This library requires Java 11 bytecode & JDK at runtime. (At build time we use current latest LTS tools.)

Copilot uses AI. Check for mistakes.
@vorburger
Copy link
Collaborator Author

Overlaps with #267 from @cstamas ...

@vorburger vorburger requested a review from cstamas January 20, 2026 22:41
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.

3 participants