Skip to content

Document CRC32 checksum value - #469

Merged
jonathanpeppers merged 1 commit into
mainfrom
jonathanpeppers-document-java-util-zip-8a0
Aug 14, 2026
Merged

jonathanpeppers merged 1 commit into
mainfrom
jonathanpeppers-document-java-util-zip-8a0

Conversation

@jonathanpeppers

Copy link
Copy Markdown
Member

Summary

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

Refs #256

Sources

Validation

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

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 generated XML documentation for Java.Util.Zip.CRC32.Value to replace a placeholder <value> description with the source-defined return value description, improving the completeness of the Java.Util.Zip API docs.

Changes:

  • Replaces the <value> placeholder for Java.Util.Zip.CRC32.Value with “The current checksum value.”
  • Leaves existing summary/remarks (including Android attribution and reference link) intact while making the property documentation non-placeholder.

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

@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 CRC32 value contract is now documented accurately.

@jonathanpeppers
jonathanpeppers merged commit 9507f4e into main Aug 14, 2026
4 checks passed
@jonathanpeppers
jonathanpeppers deleted the jonathanpeppers-document-java-util-zip-8a0 branch August 14, 2026 20:33
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