Skip to content

Conversation

@zeitlinger
Copy link
Member

No description provided.

Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
Copy link
Contributor

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 fixes GitHub Pages generation by refactoring the Hugo schema.org JSON-LD template and adds GitHub Pages build testing to the test-release-build workflow.

Changes:

  • Refactored Hugo microformats schema template from manual JSON construction to using Hugo's dict and jsonify functions
  • Added fetch-tags and fetch-depth configuration to enable git tag access in the workflow
  • Changed workflow job from building only Javadoc to building the complete GitHub Pages site

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
docs/themes/hugo-geekdoc/layouts/partials/microformats/schema.html Refactored JSON-LD structured data generation to use dict/jsonify instead of manual JSON construction, fixing quoting issues
.github/workflows/test-release-build.yml Added GitHub Pages build testing with git tag fetching support and BASE_URL configuration

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

zeitlinger and others added 2 commits February 4, 2026 13:35
…html

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
@zeitlinger zeitlinger enabled auto-merge (squash) February 4, 2026 12:37
@zeitlinger zeitlinger merged commit 586eaf5 into main Feb 4, 2026
39 checks passed
@zeitlinger zeitlinger deleted the fix-gh-pages branch February 4, 2026 12:43
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.

3 participants