Skip to content

Strip the smallest common indent from schema-row descriptions - #1799

Open
shadromani wants to merge 1 commit into
mainfrom
sromani/uhu-3499-the-markdown-build-keeps-the-nesting-of-schema-rows-with-any
Open

shadromani wants to merge 1 commit into
mainfrom
sromani/uhu-3499-the-markdown-build-keeps-the-nesting-of-schema-rows-with-any

Conversation

@shadromani

Copy link
Copy Markdown
Contributor

Fixes UHU-3499: https://linear.app/maxmind/issue/UHU-3499/the-markdown-build-keeps-the-nesting-of-schema-rows-with-any-indent

layouts/partials/markdown/schema-row.md removed exactly two leading spaces from each line of a field description. A row with a different indent lost its list nesting, or became a code block, and Hugo showed no error.

  • The partial now finds the smallest indent of the non-blank lines and removes that amount. Blank lines stay blank. A description with no indent is unchanged.

Verified with a full build: the output is identical to main. A row indented by four spaces with a nested list keeps its nesting. Lint passes.

Follow-up to the review of #1771.

🤖 Generated with Claude Code

The Markdown schema-row partial removed exactly two leading spaces from
each line of a field description. That is correct only while every row
uses a two-space base indent. A row with a different indent lost its
list nesting, or became an indented code block, and Hugo gave no error.

Find the smallest indent of the non-blank lines and remove that amount.
All current rows use a two-space or no indent, so the site output does
not change.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 27, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 43cde90e-258e-4841-930c-4b84e808f53e
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying dev-site with  Cloudflare Pages  Cloudflare Pages

Latest commit: cb38227
Status: ✅  Deploy successful!
Preview URL: https://7b2c4096.dev-site-4ua.pages.dev
Branch Preview URL: https://sromani-uhu-3499-the-markdow.dev-site-4ua.pages.dev

View logs

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant