Copilot commented on code in PR #2699:
URL: https://github.com/apache/fluss/pull/2699#discussion_r3465685447
##########
website/build_versioned_docs.sh:
##########
@@ -122,6 +122,14 @@ for branch in $branches; do
# Copy the website/docs directory to the target directory
$CP_CMD "$version_dir/" || { echo "Failed to copy for branch:
$clean_branch_name"; continue; }
echo "Copied documentation for branch '$clean_branch_name' to
'$version_dir'"
+
+ # Fix absolute links in versioned docs (convert
/docs/maintenance/filesystems/hdfs.md to relative path)
+
lakehouse_file="$version_dir/maintenance/tiered-storage/lakehouse-storage.md"
+ if [ -f "$lakehouse_file" ]; then
+ sed -i.bak
's|/docs/maintenance/filesystems/hdfs\.md|../../filesystems/hdfs|g'
"$lakehouse_file"
+ rm -f "$lakehouse_file.bak"
+ echo "Fixed absolute links in '$lakehouse_file'"
+ fi
Review Comment:
The sed replacement writes a relative link without a `.md` extension. The
rest of the docs consistently link to other docs using the `.md` filename
(e.g., `filesystems/overview.md`), and keeping the extension avoids
link-resolution differences between Markdown vs. URL path handling in the docs
toolchain.
--
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]
