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

[fix][doc] remove This example uses Pulsar x.x.x #17533

Merged
merged 7 commits into from
Sep 9, 2022

Conversation

labuladong
Copy link
Contributor

Fixes comment in #17499

Modifications

Remove redundant This example uses Pulsar x.x.x

Documentation

Check the box below or label this PR directly.

Need to update docs?

  • doc-required
    (Your PR needs to update docs and you will update later)

  • doc-not-needed
    (Please explain why)

  • doc
    (Your PR contains doc changes)

  • doc-complete
    (Docs have been already added)

cc @Anonymitaet @tisonkun @momo-jun

Copy link
Member

@tisonkun tisonkun left a comment

Choose a reason for hiding this comment

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

It's more clear to be a dedicated patch now! LGTM.

@github-actions github-actions bot added the doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. label Sep 8, 2022
Copy link
Contributor

@momo-jun momo-jun left a comment

Choose a reason for hiding this comment

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

LGTM.
@labuladong can you pls also add [fix][doc] in the PR title to align with the naming conventions?

@labuladong labuladong changed the title remove This example uses Pulsar x.x.x [fix][doc] remove This example uses Pulsar x.x.x Sep 8, 2022
@labuladong
Copy link
Contributor Author

LGTM. @labuladong can you pls also add [fix][doc] in the PR title to align with the naming conventions?

Done.

@momo-jun momo-jun added this to the 2.11.0 milestone Sep 8, 2022
@labuladong
Copy link
Contributor Author

All tests passed. Could you help to merge this pr? @Anonymitaet

@momo-jun momo-jun merged commit 9aaa8ef into apache:master Sep 9, 2022
@labuladong labuladong deleted the improve-offload-doc branch September 9, 2022 07:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Your PR contains doc changes, no matter whether the changes are in markdown or code files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants