Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update protos file, generated files, and record a new conformance test failure. #1753

Merged
merged 4 commits into from
Feb 6, 2025

Conversation

thomasvl
Copy link
Collaborator

@thomasvl thomasvl commented Feb 6, 2025

No description provided.

Using upstream cab65df75ed1b1e7a3e76b63fbae5deea228ac9f.
Upstream has revised some handing of non canonical form MessageSet encoding.
Not all upstream languages pass it yet, but it looks like upb is getting
a check to support it, so we'll want to eventually do the work to match,
but documenting the failure for now.
@thomasvl thomasvl requested a review from allevato February 6, 2025 21:47
@thomasvl thomasvl added the semver/none No version bump required. label Feb 6, 2025
@thomasvl thomasvl merged commit 683e459 into apple:main Feb 6, 2025
13 of 14 checks passed
@thomasvl thomasvl deleted the update_proto_files branch February 6, 2025 22:05
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
semver/none No version bump required.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants