Skip to content

schema markup fmt fixes#24214

Merged
dvdksn merged 2 commits intodocker:mainfrom
dvdksn:schema-markup-fmt-fixes
Feb 25, 2026
Merged

schema markup fmt fixes#24214
dvdksn merged 2 commits intodocker:mainfrom
dvdksn:schema-markup-fmt-fixes

Conversation

@dvdksn
Copy link
Contributor

@dvdksn dvdksn commented Feb 25, 2026

  • site: use array for schema.org keywords
  • site: truncate schema.org titles at 110 chars

Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
@netlify
Copy link

netlify bot commented Feb 25, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 5b3eac0
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/699ef8d901e4cf0008229a4b
😎 Deploy Preview https://deploy-preview-24214--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@dvdksn dvdksn requested a review from a team February 25, 2026 13:27
@github-actions github-actions bot added the hugo Updates related to hugo label Feb 25, 2026
@dvdksn dvdksn added status/review and removed hugo Updates related to hugo labels Feb 25, 2026
Copy link
Contributor

@docker-agent docker-agent bot left a comment

Choose a reason for hiding this comment

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

Review Summary

This PR updates the Hugo schema.org markup template with two changes:

  1. Removes the delimit function from keywords processing, allowing the keywords to be output as a JSON array instead of a comma-separated string
  2. Truncates titles to 110 characters in schema.org headline and name fields

Both changes are valid:

  • Schema.org explicitly supports arrays for keywords in JSON-LD format, so removing the delimit function is acceptable and arguably cleaner
  • The 110-character truncation aligns with Google's SEO guidelines for structured data headlines

No issues found. This PR looks good.

@dvdksn dvdksn merged commit fb91a08 into docker:main Feb 25, 2026
18 checks passed
@dvdksn dvdksn deleted the schema-markup-fmt-fixes branch February 25, 2026 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants