GabrielBBaldez opened a new pull request, #3517:
URL: https://github.com/apache/fluss/pull/3517

   ### Purpose
   
   Linked issue: close #3516
   
   The documentation website build fails on Windows because the 
`remark-version-replace` plugin does not substitute version placeholders.
   
   ### Brief change log
   
   `getDocsVersionName()` now splits the file path on both `/` and `\`, so the 
docs version is detected on Windows as well as on POSIX. Previously it split on 
`'/'` only, so on Windows (`vfile.path` uses `\`) it returned `''`, leaving 
`$FLUSS_VERSION$` / `$FLUSS_MAVEN_REPO_URL$` etc. unsubstituted and failing the 
broken-links check (~24 broken links to a literal `$FLUSS_MAVEN_REPO_URL$/...`).
   
   ### Tests
   
   Built the website locally on Windows (`npm run clear && npm run build`): 
before the fix, ~24 broken links; after, 0. No change on POSIX — the character 
class `/[/\]/` is equivalent to `/` when the path contains no backslashes.
   
   ### Generative AI disclosure
   
   - [x] Yes
   
   Generated-by: Claude Code (Opus 4.8) following the guidelines
   


-- 
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]

Reply via email to