Skip to content

Conversation

@glbrntt
Copy link
Collaborator

@glbrntt glbrntt commented Dec 4, 2025

Motivation:

A recent release of grpc-swift-2 added a few sendable annotations to the generated code. The expected code in our tests is now incorrect so CI is failing.

Modifications:

  • Bump version
  • Update test expectations
  • Regenerate

Result:

CI passes

Motivation:

A recent relase of grpc-swift-2 added a few sendable annotations to the
generated code. The expected code in our tests is now incorrect so CI is
failing.

Modifications:

- Bump version
- Update test expectations
- Regenerate

Result:

CI passes
@glbrntt glbrntt added the 🔨 semver/patch No public API change. label Dec 4, 2025
@glbrntt glbrntt enabled auto-merge (squash) December 4, 2025 13:17
Copy link
Collaborator

@rafaelcepeda rafaelcepeda left a comment

Choose a reason for hiding this comment

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

LGTM

@glbrntt glbrntt merged commit a1aa982 into grpc:main Dec 4, 2025
42 checks passed
@glbrntt glbrntt deleted the update-dep branch December 4, 2025 13:36
dongjoon-hyun added a commit to apache/spark-connect-swift that referenced this pull request Dec 19, 2025
### What changes were proposed in this pull request?

This PR aims to upgrade `gRPC Swift Protobuf` to 2.1.2.

### Why are the changes needed?

`gRPC Swift Protobuf` 2.1.2 is the first release with tested with Swift 6.2 officially. In addition, it requires the latest `grpc-swift-2` dependency (#268).
- https://github.com/grpc/grpc-swift-protobuf/releases/tag/2.1.2
  - grpc/grpc-swift-protobuf#88
  - grpc/grpc-swift-protobuf#91

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Pass the CIs.

### Was this patch authored or co-authored using generative AI tooling?

No.

Closes #270 from dongjoon-hyun/SPARK-54786.

Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔨 semver/patch No public API change.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants