Skip to content

Update ProtoBuf.jl to v1.2.2#217

Merged
comnik merged 1 commit intomainfrom
ncg-update-protobuf
Feb 25, 2026
Merged

Update ProtoBuf.jl to v1.2.2#217
comnik merged 1 commit intomainfrom
ncg-update-protobuf

Conversation

@comnik
Copy link
Collaborator

@comnik comnik commented Feb 25, 2026

Summary

  • Updates ProtoBuf.jl from v1.2.0 to v1.2.2 and regenerates protobuf code
  • v1.2.2 makes auto-generated file headers more stable by removing timestamps and absolute paths, reducing noisy diffs when regenerating protobuf code across machines or over time

Test plan

  • Julia SDK tests pass (130 test items, 3540 assertions)

🤖 Generated with Claude Code

@comnik comnik requested review from davidwzhao and nystrom February 25, 2026 01:18
nystrom
nystrom approved these changes Feb 25, 2026
View reviewed changes
Copy link
Contributor

@nystrom nystrom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@comnik comnik merged commit 7033768 into main Feb 25, 2026
6 checks passed
@comnik comnik deleted the ncg-update-protobuf branch February 25, 2026 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nystrom nystrom nystrom approved these changes

@davidwzhao davidwzhao Awaiting requested review from davidwzhao

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@comnik @nystrom