refactor: vendor PaperMod theme directly instead of using Hugo module#2
refactor: vendor PaperMod theme directly instead of using Hugo module#2
Conversation
There was a problem hiding this comment.
Pull request overview
This PR refactors the project to vendor the PaperMod theme directly instead of using it as a Hugo module dependency. This provides full control over theme code and eliminates external dependencies.
Changes:
- Removed Hugo module dependency on PaperMod from go.mod, go.sum, and hugo.yml
- Added PaperMod layouts (shortcodes, partials, templates) directly to the repository
- Added JavaScript files (fastsearch.js, fuse.basic.min.js, license.js) for search functionality
- Added i18n translation file (en.yaml)
- Added LICENSES directory documenting PaperMod (MIT) and Fuse.js (Apache 2.0) licenses
- Updated CLAUDE.md documentation to reflect new structure
Reviewed changes
Copilot reviewed 28 out of 30 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| go.mod | Removed PaperMod module dependency |
| go.sum | Removed module checksums (file deleted) |
| config/_default/hugo.yml | Removed module import configuration |
| layouts/shortcodes/*.html | Added theme shortcodes (rtl, ltr, rawhtml, figure, inTextImg, collapse) |
| layouts/partials/*.html | Added theme partials (breadcrumbs, cover, home_info, comments, etc.) |
| layouts/partials/templates/*.html | Added OpenGraph, Twitter Cards, and Schema.org JSON-LD templates |
| layouts/partials/templates/_funcs/get-page-images.html | Added image processing helper function |
| layouts/_default/_markup/render-image.html | Added custom image renderer for markdown |
| assets/js/fastsearch.js | Added search implementation using Fuse.js |
| assets/js/fuse.basic.min.js | Added Fuse.js fuzzy search library |
| assets/js/license.js | Added license information for JavaScript files |
| i18n/en.yaml | Added English translation strings |
| LICENSES/PAPERMOD.md | Documented PaperMod MIT license |
| LICENSES/FUSEJS.md | Documented Fuse.js Apache 2.0 license |
| CLAUDE.md | Updated documentation to reflect vendored theme structure |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 28 out of 30 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
cboone
force-pushed
the
claude/remove-papermod-dependency-g0TyG
branch
from
302aa2b to
42c5ca0
Compare
cboone
force-pushed
the
claude/remove-papermod-dependency-g0TyG
branch
from
42c5ca0 to
e538dcd
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 28 out of 30 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 29 out of 31 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Remove the Hugo module dependency on PaperMod and include all theme files directly in the repository. This gives full control over the theme code and eliminates the external dependency. Changes: - Copy PaperMod layouts, JavaScript, and i18n files into repository - Remove module configuration from go.mod and hugo.yml - Add LICENSES directory with PaperMod (MIT) and Fuse.js (Apache 2.0) licenses - Update CLAUDE.md to reflect new project structure https://claude.ai/code/session_01ACdKrQrVijTD7DippVriNm
- Fix partial path consistency in opengraph.html - Fix HTML boolean attribute syntax in collapse.html - Add missing semicolon in fastsearch.js
- Add missing space in figure.html pipe syntax - Fix string comparison for openByDefault in collapse.html - Remove ineffective document.activeElement assignment - Use for-of loop instead of for-in for array iteration
cboone
force-pushed
the
claude/remove-papermod-dependency-g0TyG
branch
from
62368ab to
0e6230a
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 29 out of 31 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Replace loose equality (==, !=) with strict equality (===, !==) for DOM element comparisons to follow JavaScript best practices.
Remove the Hugo module dependency on PaperMod and include all theme files
directly in the repository. This gives full control over the theme code and
eliminates the external dependency.
Changes:
https://claude.ai/code/session_01ACdKrQrVijTD7DippVriNm