Skip to content

Fix Orleans.Serialization.Protobuf README to document Google.Protobuf instead of protobuf-net#9779

Merged
ReubenBond merged 2 commits into
mainfrom
copilot/fix-readme-orleans-serialization
Feb 15, 2026
Merged

Fix Orleans.Serialization.Protobuf README to document Google.Protobuf instead of protobuf-net#9779
ReubenBond merged 2 commits into
mainfrom
copilot/fix-readme-orleans-serialization

Conversation

Copilot AI commented Nov 16, 2025

Copy link
Copy Markdown
Contributor
  • Review the current README and identify incorrect information
  • Update the README to show Google.Protobuf usage instead of protobuf-net
  • Update example code to demonstrate .proto file usage
  • Update example code to show proper Google.Protobuf attributes and patterns
  • Verify the documentation is consistent with the actual implementation
  • Build the project to ensure no issues
  • Streamline README by referencing Microsoft's official Protobuf documentation
  • Complete the task

Summary

Updated the README to be more concise and reference Microsoft's official documentation:

Changes:

  • Made introduction more concise
  • Replaced detailed .proto file and project configuration instructions with reference to Create Protobuf messages for .NET apps
  • Simplified example code, removing redundant explanations
  • Consolidated collections section into a brief note
  • Added official Microsoft Protobuf guide to documentation section
  • Removed generic Protocol Buffers documentation link in favor of .NET-specific guidance

This follows the feedback to "ground the document by reading the Microsoft documentation and incorporating it" while removing unnecessary details that are already covered in the official guide.

Original prompt

On Orleans.Serialization.Protobuf, Fix this README to align with reality - for example, note that this adds support for Google.Protobuf, but the README currently shows attributes for a different package.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Microsoft Reviewers: Open in CodeFlow

Copilot AI changed the title [WIP] Fix README to align with Orleans.Serialization.Protobuf details Fix Orleans.Serialization.Protobuf README to document Google.Protobuf instead of protobuf-net Nov 16, 2025
Copilot AI requested a review from ReubenBond November 16, 2025 18:59
@ReubenBond

Copy link
Copy Markdown
Member

@copilot Ground the document by reading https://learn.microsoft.com/en-us/aspnet/core/grpc/protobuf?view=aspnetcore-9.0 and incorporating it. Ideally, reference it so that we can remove unnecessary details

Copilot AI commented Nov 16, 2025

Copy link
Copy Markdown
Contributor Author

@copilot Ground the document by reading https://learn.microsoft.com/en-us/aspnet/core/grpc/protobuf?view=aspnetcore-9.0 and incorporating it. Ideally, reference it so that we can remove unnecessary de...

Done. The README now references the Microsoft documentation and removes the detailed setup instructions that are covered there. See commit 4360f06.

@ReubenBond ReubenBond marked this pull request as ready for review November 18, 2025 04:34
Copilot AI review requested due to automatic review settings November 18, 2025 04:34

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 Orleans.Serialization.Protobuf README to correctly document Google.Protobuf integration instead of protobuf-net, which was incorrectly referenced in the original documentation.

Key changes:

  • Updated introduction to explicitly mention Google.Protobuf package
  • Streamlined documentation by referencing Microsoft's official Protobuf guide instead of duplicating setup instructions
  • Simplified example code and removed protobuf-net specific attributes

@ReubenBond ReubenBond enabled auto-merge February 15, 2026 13:41
Copilot AI and others added 2 commits February 15, 2026 05:41
Co-authored-by: ReubenBond <203839+ReubenBond@users.noreply.github.com>
Co-authored-by: ReubenBond <203839+ReubenBond@users.noreply.github.com>
@ReubenBond ReubenBond force-pushed the copilot/fix-readme-orleans-serialization branch from e244581 to 786bafb Compare February 15, 2026 13:41
@ReubenBond ReubenBond added this pull request to the merge queue Feb 15, 2026
Merged via the queue into main with commit 757b53e Feb 15, 2026
59 checks passed
@ReubenBond ReubenBond deleted the copilot/fix-readme-orleans-serialization branch February 15, 2026 14:59
rkargMsft pushed a commit to rkargMsft/orleans that referenced this pull request Feb 27, 2026
… instead of protobuf-net (dotnet#9779)

* Fix Orleans.Serialization.Protobuf README to document Google.Protobuf

Co-authored-by: ReubenBond <203839+ReubenBond@users.noreply.github.com>

* Streamline README by referencing Microsoft Protobuf documentation

Co-authored-by: ReubenBond <203839+ReubenBond@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ReubenBond <203839+ReubenBond@users.noreply.github.com>
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 18, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants