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 the proto files from upstream. #1537

Merged
merged 2 commits into from
Jan 8, 2024

Conversation

thomasvl
Copy link
Collaborator

@thomasvl thomasvl commented Jan 8, 2024

No description provided.

@thomasvl thomasvl requested a review from allevato January 8, 2024 15:35
@thomasvl
Copy link
Collaborator Author

thomasvl commented Jan 8, 2024

protoc is doing something I didn't expect here. it seems the files labeled with editions are still getting through the generator even though we don't support editions.

Using 991a2f4d54c6944396f55eaf13ae5248598ed28a from protobuf.

Leaving off the file have been converted over to editions.
@thomasvl thomasvl force-pushed the update_upstream_protos branch from 9062552 to 1637f60 Compare January 8, 2024 16:17
@thomasvl
Copy link
Collaborator Author

thomasvl commented Jan 8, 2024

Updated commits to not update the files that have moved to editions, reached out to the protobuf team to check on the behavior change I'm seeing.

@thomasvl thomasvl merged commit da0566a into apple:main Jan 8, 2024
9 checks passed
@thomasvl thomasvl deleted the update_upstream_protos branch January 8, 2024 16:34
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants