Skip to content

build: migrate Markdown tooling to rumdl - #25710

Open
dvdksn wants to merge 2 commits into
docker:mainfrom
dvdksn:codex/migrate-rumdl
Open

build: migrate Markdown tooling to rumdl#25710
dvdksn wants to merge 2 commits into
docker:mainfrom
dvdksn:codex/migrate-rumdl

Conversation

@dvdksn

@dvdksn dvdksn commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Summary

Replace markdownlint and Markdown-specific Prettier usage with pinned rumdl tooling and a native Hugo-aware configuration. Retain Prettier for non-Markdown files and update repository workflows and guidance accordingly.

Generated by Codex

dvdksn and others added 2 commits August 4, 2026 10:38
Markdown linting and formatting used separate markdownlint and Prettier paths. Replace Markdown-specific tooling with pinned rumdl native config while retaining Prettier for non-Markdown files.

Co-Authored-By: Claude <noreply@anthropic.com>
AGENTS.md incorrectly stated that rumdl requires the keywords front matter field. Remove the false validator claim while retaining keywords as a repository requirement.

Co-Authored-By: Claude <noreply@anthropic.com>
@netlify

netlify Bot commented Aug 4, 2026

Copy link
Copy Markdown

Deploy Preview for docsdocker ready!

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

@github-actions github-actions Bot added area/engine Issue affects Docker engine/daemon area/swarm Issue affects swarm and clustering functionality of docker area/get-started Relates to get started and onboarding docs area/release Relates to CI or deployment hugo Updates related to hugo dependencies Vendoring, packaging updates area/config area/tests area/guides labels Aug 4, 2026
@dvdksn dvdksn added the status/review Pull requests that are ready for review label Aug 4, 2026
@dvdksn
dvdksn requested a review from a team August 4, 2026 11:32

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

This PR is a clean mechanical migration of the Markdown linting toolchain from markdownlint/Prettier to rumdl. All changes are confined to build infrastructure, tooling configuration, and agent prompt files — no documentation content is modified.

The changes are consistent and correct:

  • Dockerfile replaces the markdownlint image with the pinned rumdl image
  • Scripts update tool invocations from npx markdownlint-cli to npx --no-install rumdl check/fmt
  • Agent YAML files update references from markdownlint to rumdl
  • .prettierignore correctly excludes **/*.md so Prettier no longer processes Markdown

No documentation content issues found. The tooling migration looks straightforward and well-scoped.

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

This PR cleanly migrates Markdown tooling from markdownlint/Prettier to rumdl. The exclusions previously passed as CLI flags to markdownlint (--ignore content/manuals/engine/release-notes/ etc.) are correctly replicated in .rumdl.toml's exclude block, so CI lint coverage is preserved. rumdl is distributed as a proper npm package (package.json devDependencies + platform-specific @rumdl/cli-* binaries), making the npx --no-install rumdl invocations throughout the codebase correct. No documentation content issues, vendored-file edits, missing redirects, or AI-isms were found.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/config area/engine Issue affects Docker engine/daemon area/get-started Relates to get started and onboarding docs area/guides area/release Relates to CI or deployment area/swarm Issue affects swarm and clustering functionality of docker area/tests dependencies Vendoring, packaging updates hugo Updates related to hugo status/review Pull requests that are ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants