Skip to content

Document checksum value property - #466

Merged
jonathanpeppers merged 2 commits into
mainfrom
jonathanpeppers-document-java-util-zip
Aug 14, 2026
Merged

jonathanpeppers merged 2 commits into
mainfrom
jonathanpeppers-document-java-util-zip

Conversation

@jonathanpeppers

Copy link
Copy Markdown
Member

Summary

Documents the source-defined return value for Java.Util.Zip.IChecksum.Value.

Refs #256

Sources

Validation

  • XML parsing: passed for docs/xml/Java.Util.Zip/IChecksum.xml.
  • CRLF validation: passed (292 lines; no bare LF line endings).
  • Scoped placeholder count: 0 for Java.Util.Zip.IChecksum.Value.
  • git diff --check: passed.

Refs #256

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown

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 updates the XML documentation for Java.Util.Zip.IChecksum.Value to replace the “To be added.” placeholder and align the docs with the Android reference description of Checksum.getValue().

Changes:

  • Updates the <summary> for Java.Util.Zip.IChecksum.Value to describe the property’s meaning.
  • Replaces the placeholder <value> text with a concrete description.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/xml/Java.Util.Zip/IChecksum.xml Outdated
Refs #256

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@jonathanpeppers

Copy link
Copy Markdown
Member Author

@dalexsoto review

@dalexsoto dalexsoto left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The checksum value contract is now documented accurately and consistently with the source API.

@jonathanpeppers
jonathanpeppers merged commit 9aa66f9 into main Aug 14, 2026
3 checks passed
@jonathanpeppers
jonathanpeppers deleted the jonathanpeppers-document-java-util-zip branch August 14, 2026 20:32
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