Skip to content

add example of using Pulsar with testcontainers-go#597

Merged
mdelapenya merged 4 commits intomainfrom
docs/pulsar-example
Oct 31, 2022
Merged

add example of using Pulsar with testcontainers-go#597
mdelapenya merged 4 commits intomainfrom
docs/pulsar-example

Conversation

@clive-jevons
Copy link
Copy Markdown
Contributor

@clive-jevons clive-jevons commented Oct 28, 2022

A quick example of how I used Pulsar in a recent test for another project which others might find useful.

It essentially mimics the way the testcontainers-java Pulsar module sets up Pulsar.

@clive-jevons clive-jevons added the documentation Docs, docs, docs. label Oct 28, 2022
@clive-jevons clive-jevons self-assigned this Oct 28, 2022
@clive-jevons clive-jevons requested a review from a team as a code owner October 28, 2022 18:24
@mdelapenya
Copy link
Copy Markdown
Member

This is super cool!! Thanks for adding the example.

I'd remove the gomega dependency to avoid associating the example with a test framework. I'd use testing library + require or assert. You can check the ones used in the project as a reference.

In any case, this LGTM!!

@mdelapenya
Copy link
Copy Markdown
Member

I've just realized that the example is not shown in the website because the mkdocs descriptor living in the root for of the project, she's not included the new entry. Could you please add it?

@clive-jevons
Copy link
Copy Markdown
Contributor Author

This is super cool!! Thanks for adding the example.

I'd remove the gomega dependency to avoid associating the example with a test framework. I'd use testing library + require or assert. You can check the ones used in the project as a reference.

In any case, this LGTM!!

done 👍

... just saw that I had the log-following code still in there ... do you think we should leave that in the example, or should I remove that so it's just showing Pulsar usage?

@clive-jevons
Copy link
Copy Markdown
Contributor Author

I've just realized that the example is not shown in the website because the mkdocs descriptor living in the root for of the project, she's not included the new entry. Could you please add it?

done 👍

Copy link
Copy Markdown
Member

@mdelapenya mdelapenya left a comment

Choose a reason for hiding this comment

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

LGTM!!

I've test it locally with success, thanks for this great example on Apache Pulsar usage. Great job

@mdelapenya mdelapenya merged commit 212978a into main Oct 31, 2022
@mdelapenya mdelapenya deleted the docs/pulsar-example branch October 31, 2022 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Docs, docs, docs.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants