rmannibucau commented on code in PR #78: URL: https://github.com/apache/karaf-site/pull/78#discussion_r4026728628
########## README.md: ########## @@ -48,6 +48,8 @@ npm run watch:css # rebuild CSS continuously while styling npm run optimize:svg # minify the SVG assets (logos) after editing any images/*.svg ``` +`assets/css/karaf.css` is committed to the repository, not gitignored: the production deploy below only runs `bundle exec jekyll build`, which cannot resolve the Bootstrap 5 Sass imports in `_scss/` (that requires `node_modules` via the `sass` CLI). Whenever you change anything under `_scss/`, run `npm run build:css` and commit the regenerated `assets/css/karaf.css` together with your source change. Review Comment: [think sonnet was dra](https://github.com/apache/karaf-site/pull/78/changes#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519R10) was intended to solve it, can need to adjust the workflow but commiting the build outputs doesn't sound wise, even if I respect a lot Claude -- 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]
