Skip to content

Conversation

@momo-jun
Copy link
Contributor

@momo-jun momo-jun commented Feb 11, 2022

Modifications

To align with #14225 , the following change has been made:

Change the pulsar-admin topics *-schema-compatibility-strategy command to the pulsar-admin topicsPolicies *-schema-compatibility-strategy command.

Doc Site Preview

image

Documentation

  • doc

    (If this PR contains doc changes)

@github-actions
Copy link

@momo-jun:Thanks for your contribution. For this PR, do we need to update docs?
(The PR template contains info about doc, which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)

@github-actions
Copy link

@momo-jun:Thanks for providing doc info!

@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 Feb 11, 2022
@momo-jun
Copy link
Contributor Author

Hi @nodece @Anonymitaet, can you please review this doc PR? Thanks.

Copy link
Member

@Anonymitaet Anonymitaet left a comment

Choose a reason for hiding this comment

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

Thanks.
Approve from the tech writing pespective.

Copy link
Member

@nodece nodece left a comment

Choose a reason for hiding this comment

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

LGTM

@momo-jun momo-jun changed the title [Doc] [Do-not-Merge] Move schema compatibility strategy CMD from topics to topicsPolicies [Doc] Move schema compatibility strategy CMD from topics to topicsPolicies Feb 16, 2022
@codelipenghui codelipenghui merged commit e3c9684 into apache:master Feb 16, 2022
Nicklee007 pushed a commit to Nicklee007/pulsar that referenced this pull request Apr 20, 2022
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.

4 participants