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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
