Update module github.com/google/docsy/theme to v0.17.0 - #2825
Merged
Merged
Conversation
✅ Deploy Preview for selenium-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Docsy 0.17.0 switched its default Sass transpiler from Hugo's built-in LibSass to Dart Sass, which broke the build with "no Dart Sass binary found in $PATH". Per Docsy's upgrade guide, add the sass-embedded package (Docsy-tested at 1.102.0) and regenerate packages/hugoautogen via `hugo mod npm pack` to pick up the theme's other dependency bumps (e.g. Font Awesome 7.3.1). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Contributor
Author
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
Two more gaps beyond the sass-embedded dependency itself:
- build-site.sh invokes hugo directly (not via an npm script), so
node_modules/.bin was never on $PATH, and Hugo's Dart Sass transpiler
resolves its "sass" binary from $PATH.
- Even with node_modules/.bin on PATH, sass-embedded's own optional
"sass" fallback dependency claims the same node_modules/.bin/sass
bin name, and that fallback refuses embedded-protocol mode ("sass
--embedded is unavailable in pure JS mode"), which Hugo needs. Build
a dedicated PATH entry pointing at sass-embedded's own script so it
wins over the ambiguous node_modules/.bin symlink.
- hugo.toml's security.exec.allow was a site-specific override that
only allowed the old "dart-sass-embedded" binary name; add "sass",
the name the current package ships.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
link-check.yml's htmltest job invokes hugo directly (not through build-site.sh), so it hit the same "no Dart Sass binary found in $PATH" failure test_build had. Extract the PATH setup into scripts/setup-dart-sass-path.sh so both call sites share it instead of duplicating the logic. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
diemol
approved these changes
Sep 16, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR contains the following updates:
v0.16.0→v0.17.0Release Notes
google/docsy (github.com/google/docsy/theme)
v0.17.0Compare Source
Release summary
What's Changed
latestby @chalin in #2704New Contributors
Full Changelog: docsy/docsy@v0.16.0...v0.17.0
Configuration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.