Skip to content

Use shared _head template from goblog#2

Merged
compscidr merged 1 commit intomainfrom
feature/shared-head-template
Mar 19, 2026
Merged

Use shared _head template from goblog#2
compscidr merged 1 commit intomainfrom
feature/shared-head-template

Conversation

@compscidr
Copy link
Copy Markdown
Contributor

Summary

  • Replace the duplicated <head> section (79 lines) with {{ template "_head" . }}
  • Depends on Extract shared head template from themes goblog#526 which extracts all <head> content into a shared template
  • Meta tags, OG, structured data, CDN imports, and settings like robots_tag and site_url are now managed centrally

Test plan

  • Deploy with updated goblog and verify pages render correctly
  • Verify meta tags appear in page source

🤖 Generated with Claude Code

Replace the duplicated <head> section with {{ template "_head" . }}
from goblog's shared templates. Meta tags, OG, structured data, CDN
imports, and settings like robots_tag and site_url are now managed
centrally in goblog.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings March 19, 2026 18:17
@compscidr compscidr merged commit 55ed5ec into main Mar 19, 2026
1 check passed
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the theme to use a shared _head template (provided by goblog) instead of maintaining a large, duplicated <head> block locally, aiming to centralize meta tags, structured data, and asset imports.

Changes:

  • Replace the theme’s inline <head> HTML in templates/header.html with {{ template "_head" . }}.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

{{ end }}{{ end }}
{{ with .plugin_head_html }}{{ . | rawHTML }}{{ end }}
</head>
{{ template "_head" . }}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants