bamaer commented on PR #8905: URL: https://github.com/apache/hop/pull/8905#issuecomment-6161735002
Docs-only, so nothing here can break anything. **Worth fixing** - hop-website already has `run-website-local.sh` for this: `npm run build:hop` and then `npm run serve`, plus `--fast` to re-render only the docs after a first full build. Pointing to that script would shorten the section to "check out hop-website next to `hop`, run `./run-website-local.sh`". It would also drop the manual `cp -R static/js public/` step, because the Astro build should populate `public/js`. The `latest`-not-`next` note is worth keeping. Verified: the documented steps build cleanly against hop-website `main`. I haven't run `run-website-local.sh` myself. -- 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]
