Skip to content

Style cleanup: add missing keywords and remove bold-for-emphasis#25127

Open
brandonh6k wants to merge 7 commits into
docker:mainfrom
brandonh6k:bh/guides-style-cleanup
Open

Style cleanup: add missing keywords and remove bold-for-emphasis#25127
brandonh6k wants to merge 7 commits into
docker:mainfrom
brandonh6k:bh/guides-style-cleanup

Conversation

@brandonh6k
Copy link
Copy Markdown
Contributor

Two related style-conformance fixes across content/guides/, motivated by a recent audit of guide-page style against STYLE.md, COMPONENTS.md, and AGENTS.md.

What changed

1. Add missing keywords: frontmatter to 104 guide pages

keywords is documented as required (AGENTS.md, COMPONENTS.md) but was missing from 10 single-file guides, 8 _index.md landing pages, and 86 chapter pages. Each page now has a meaningful, page-specific comma-separated keyword list derived from its title, lead content, and sibling pages where applicable.

Examples of the convention applied:

  • content/guides/testcontainers-java-spring-boot-kafka/create-project.mdtestcontainers, java, spring boot, kafka, mysql, project setup
  • content/guides/docker-compose/why.mddocker compose, multi-container, yaml, services, orchestration, application

Post-change verification confirms zero guide pages under content/guides/ are missing the field.

2. Remove bold-for-emphasis from 14 guide pages

STYLE.md restricts bold to UI elements only — never for emphasis, product names, command names, or the **Term**: description subsection-label pattern. Cleaned up the top-15 offender files identified by raw bold-span count (code blocks excluded). One file (jupyter.md, 32 spans) was untouched because all spans were legitimate UI references.

Shape of the work across 14 files:

  • ~60 **Term**: description list items de-bolded
  • ~10 identifiers/values/commands converted to backticks (e.g. **my-mysql**`my-mysql`)
  • ~10 product/emphasis bolds removed
  • 6 bolded link texts un-bolded
  • 3 blockquote-style notes converted to > [!NOTE] callouts

Legitimate UI bolds (button names, menu items, field labels, tab names) were preserved throughout. databases.md kept 47 of 49 spans for this reason — only port-mapping value and container name were converted to backticks.

Out of scope

The audit surfaced other items not addressed here:

  • Series params block missing on all 48 _index.md landing pages — this is a "decide what's authoritative" question that warrants discussion before code changes.
  • Vale prose violations (just, easily, we, etc.) — concentrated in ~5 files; can land separately.
  • Heading numbering convention (## Step N: vs ## N. vs plain) — undocumented; should be pinned down in STYLE.md or COMPONENTS.md before mechanical sweeps.

Verification

  • markdownlint (via scripts/lint.sh) clean on all changed files.
  • Vale not installed locally; relying on CI to validate prose.
  • git diff --stat confirms scope: 118 unique files, all under content/guides/. No vendored, layout, or generated-file changes.

@netlify
Copy link
Copy Markdown

netlify Bot commented May 21, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit c710974
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/6a0e82e678e1c60008fbfeaa
😎 Deploy Preview https://deploy-preview-25127--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.

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.

1 participant