Skip to content

Comments

Upgrade go pulsar client 0.14 and go version 1.21#136

Merged
maxsxu merged 3 commits intostreamnative:masterfrom
klevy-toast:upgrade-pulsar-client
Jan 10, 2025
Merged

Upgrade go pulsar client 0.14 and go version 1.21#136
maxsxu merged 3 commits intostreamnative:masterfrom
klevy-toast:upgrade-pulsar-client

Conversation

@klevy-toast
Copy link
Contributor

(If this PR fixes a github issue, please add Fixes #<xyz>.)

Fixes #135

Motivation

I am attempting to implement a feature that requires a newer client version.

Modifications

Upgrading to the latest version of the Go Pulsar Client 0.14.0, which supports minimum Go version 1.21

Verifying this change

  • Make sure that the change passes the CI checks.

This change is a trivial rework / code cleanup without any test coverage.

Documentation

Check the box below.

Need to update docs?

  • doc-required

    (If you need help on updating docs, create a doc issue)

  • no-need-doc

    (Please explain why)

  • doc

    (If this PR contains doc changes)

@klevy-toast klevy-toast requested a review from a team as a code owner January 8, 2025 00:56
@github-actions github-actions bot added the doc This pr contains a document label Jan 8, 2025
}

func testPulsarTenantExists(tenant string) resource.TestCheckFunc {
func testPulsarTenantExists() resource.TestCheckFunc {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This change is necessitated by the unparam linter..

Copy link
Member

@maxsxu maxsxu left a comment

Choose a reason for hiding this comment

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

@klevy-toast Thanks for your contributions! 👍

@maxsxu maxsxu merged commit 104bc0a into streamnative:master Jan 10, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc This pr contains a document

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Upgrade Pulsar Client to 0.14.0

3 participants