-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[improve][doc] Specially handle Python and C++ API documents #18477
Conversation
Signed-off-by: tison <[email protected]>
Signed-off-by: tison <[email protected]>
Verified the replace logics locally. Merging... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
| Language | Documentation | Release note | Code repo | | ||
| --------- |------------------------------------------------------------------------| --------------------------------------------------------------------------------- | --------------------------------------------------------------------- | | ||
| Java | [User doc](client-libraries-java.md) <br/> [API doc](/api/client/) | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client) | | ||
| C++ | [User doc](client-libraries-cpp.md) <br/> [API doc](/api/cpp/3.0.0) | [Bundled](/release-notes/) | [Standalone](https://github.com/apache/pulsar/pulsar-client-cpp) | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Tison may I know why the latest version number is hard-coded here for C++ API docs while @pulsar:apidoc:cpp@
is used in other occurrences?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See the description at apache/pulsar-site#291.
Now they have their own repository and release cycle, it's impossible to infer the proper version from the current Pulsar version. We should instead hardcode the corresponding versions for new versions.
This is the counterpart of apache/pulsar-site#291.
Now they have their own repository and release cycle, it's impossible to infer the proper version from the current Pulsar version. We should instead hardcode the corresponding versions for new versions.
Documentation
doc
doc-required
doc-not-needed
doc-complete
Matching PR in forked repository
PR in forked repository: