Repository navigation
Conversation
Signed-off-by: Muhamad Kurniawan Fauzi <krnwnfauzi19@gmail.com>
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Cache Hugo-generated resources .github/workflows/build-docs-preview.yml |
The workflow adds a cache for resources/_gen on events other than closed. Its key includes runner OS, dependency and build-input hashes, and a content hash. Restore keys match the runner OS and dependency-hash prefix. |
Priority:
Estimated code review effort: 2 (Simple) | ~8 minutes
Change: Feature
Possibly related issues
- layer5io/layer5-academy issue 281 includes caching
resources/_genin the GitHub Actions workflow. This change adds that cache to the docs preview workflow, but does not address the issue’s WebP conversion or content image mount work.
Suggested reviewers: banana-three-join
Merge Risk: 
Minimal · up to 963b7
-
No actionable merge-blocking risk is identified for the preview cache change. Normal workflow checks should complete before merging.
Pre-merge checks: 
5
-
Full details: Docstring Coverage
-
Explanation
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
-
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
Notes for Reviewers
Summary
This PR adds caching for Hugo-generated resources (
resources/_gen) to the docs preview workflow.Changes
Before / After
resources/_gencache configuredValidation
academy-theme#252.git diff --checkpassed.Related issue: Part of #281.
WebP conversion is implemented separately in layer5io/academy-theme#252. This PR only covers preview CI caching, not the downstream theme update.
Signed commits
Summary by CodeRabbit