Skip to content

Document ProtoOutputStream field tag shift - #462

Merged
jonathanpeppers merged 1 commit into
mainfrom
jonathanpeppers-document-android-util-proto
Aug 14, 2026
Merged

Document ProtoOutputStream field tag shift#462
jonathanpeppers merged 1 commit into
mainfrom
jonathanpeppers-document-android-util-proto

Conversation

@jonathanpeppers

Copy link
Copy Markdown
Member

Refs #238

Documents ProtoOutputStream.FieldIdShift from the public Android API reference. The remaining namespace placeholders are deliberately outside this narrow source-backed increment.

Sources

Validation

  • Parsed docs/xml/Android.Util.Proto/ProtoOutputStream.xml with PowerShell XML parsing.
  • Confirmed CRLF-only line endings and a focused one-member diff.

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

Updates the public Android API reference XML documentation for Android.Util.Proto.ProtoOutputStream.FieldIdShift to replace the placeholder text with source-backed documentation, as part of the incremental cleanup tracked in #238.

Changes:

  • Replaced the To be added. placeholder summary/remarks for FieldIdShift with a concrete summary describing the constant’s purpose.

💡 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 ProtoOutputStream field-tag shift documentation accurately reflects the managed constant and platform encoding.

@jonathanpeppers
jonathanpeppers merged commit 9349bd9 into main Aug 14, 2026
4 checks passed
@jonathanpeppers
jonathanpeppers deleted the jonathanpeppers-document-android-util-proto branch August 14, 2026 23:59
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