You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@paimon.apache.org by "lucasfang (via GitHub)" <gi...@apache.org> on 2026/07/09 09:46:09 UTC
[PR] add Paimon CPP docs for build docs workflow [paimon-website]
lucasfang opened a new pull request, #35:
URL: https://github.com/apache/paimon-website/pull/35
(no comment)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] docs: add Paimon CPP docs for build docs workflow [paimon-website]
Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi commented on PR #35:
URL: https://github.com/apache/paimon-website/pull/35#issuecomment-4923778993
+1
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] docs: add Paimon CPP docs for build docs workflow [paimon-website]
Posted by "Copilot (via GitHub)" <gi...@apache.org>.
Copilot commented on code in PR #35:
URL: https://github.com/apache/paimon-website/pull/35#discussion_r3550584110
##########
.github/workflows/build.yml:
##########
@@ -144,6 +144,23 @@ jobs:
mkdir -p docs/rust
cp -r /tmp/paimon-rust-site/* docs/rust/
+ - name: Clone Paimon CPP repo
+ run: git clone --depth 1 https://github.com/apache/paimon-cpp.git /tmp/paimon-cpp
+
+ - name: Build Paimon CPP docs
+ run: |
+ sudo apt-get update
+ sudo apt-get install -y doxygen
+ pip install -r /tmp/paimon-cpp/docs/requirements.txt
Review Comment:
Use `python -m pip` instead of invoking `pip` directly to ensure the installs use the interpreter provisioned by `actions/setup-python` (avoids PATH/pip-version mismatches on runners).
##########
.github/workflows/build.yml:
##########
@@ -144,6 +144,23 @@ jobs:
mkdir -p docs/rust
cp -r /tmp/paimon-rust-site/* docs/rust/
+ - name: Clone Paimon CPP repo
+ run: git clone --depth 1 https://github.com/apache/paimon-cpp.git /tmp/paimon-cpp
Review Comment:
The clone relies on the repo's default branch. Being explicit about the intended branch makes the workflow more robust if the default branch name ever changes (while still tracking the latest on that branch).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] docs: add Paimon CPP docs for build docs workflow [paimon-website]
Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi merged PR #35:
URL: https://github.com/apache/paimon-website/pull/35
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org