Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Change a stray GRPCSendable to Sendable #1588

Merged
merged 1 commit into from
Apr 14, 2023
Merged

Change a stray GRPCSendable to Sendable #1588

merged 1 commit into from
Apr 14, 2023

Conversation

glbrntt
Copy link
Collaborator

@glbrntt glbrntt commented Apr 14, 2023

Motivation:

Occurrences of GRPCSendable were replaced with Sendable in #1584 as we dropped support for 5.5. However, one was missed.

Modifications:

  • Change last GRPCSendable to Sendable

Result:

Fewer warnings.

Motivation:

Occurrences of GRPCSendable were replaced with Sendable in grpc#1584 as we
dropped support for 5.5. However, one was missed.

Modifications:

- Change last `GRPCSendable` to `Sendable`

Result:

Fewer warnings.
@glbrntt glbrntt added the semver-patch • Requires a SemVer Patch version change label Apr 14, 2023
@Lukasa Lukasa merged commit 87fa9ed into grpc:main Apr 14, 2023
@glbrntt glbrntt deleted the gb-warn branch April 14, 2023 16:04
WendellXY pushed a commit to sundayfun/grpc-swift that referenced this pull request Aug 24, 2023
Motivation:

Occurrences of GRPCSendable were replaced with Sendable in grpc#1584 as we
dropped support for 5.5. However, one was missed.

Modifications:

- Change last `GRPCSendable` to `Sendable`

Result:

Fewer warnings.
pinlin168 pushed a commit to sundayfun/grpc-swift that referenced this pull request Aug 24, 2023
Motivation:

Occurrences of GRPCSendable were replaced with Sendable in grpc#1584 as we
dropped support for 5.5. However, one was missed.

Modifications:

- Change last `GRPCSendable` to `Sendable`

Result:

Fewer warnings.
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver-patch • Requires a SemVer Patch version change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants