cleanup cli docs generation: BUILD_ALL_VERSION=1 - #273
Conversation
Signed-off-by: tison <wander4096@gmail.com>
| Usage | ||
|
|
||
| ```bash | ||
| broker-tool command |
There was a problem hiding this comment.
It seems that we remove this utility for a long time.
cc @merlimat @codelipenghui @nicoloboschi do you have ideas of the specific commit we remove this tool? I'm thinking of which versions of docs we should do the corresponding changes.
|
|
||
| # copy CLI tools docs | ||
| tools="bookkeeper pulsar-daemon pulsar-shell" | ||
| for tool in $tools | ||
| do | ||
| cp "$WEBSITE/docs/reference-cli-$tool.md" "$VERSIONED_DIR/$tool/$tool.md" | ||
| done |
There was a problem hiding this comment.
This code snippet along with the existence of the reference-cli-* files causes the current https://pulsar.apache.org/reference/#/next/bookkeeper/bookkeeper page still have a wrongly parsed yaml header.
We need to remove all this stuff to finish what @Anonymitaet proposed at apache/pulsar#18101
There was a problem hiding this comment.
@SignorMercurio would you pls help to double-check it? thanks.
There was a problem hiding this comment.
Yes, this code snippet is no longer needed and should be removed. So does reference-cli-*.md. Thanks @tisonkun!
|
ping @Anonymitaet @urfreespace as a reminder. I think this patch is ready to merge. |
4 participants
Signed-off-by: tison wander4096@gmail.com