Skip to content

Explain root categories in the YAML format reference - #537

Merged
timtebeek merged 1 commit into
masterfrom
tim/root-category-docs
Aug 28, 2026
Merged

Explain root categories in the YAML format reference#537
timtebeek merged 1 commit into
masterfrom
tim/root-category-docs

Conversation

@timtebeek

@timtebeek timtebeek commented Aug 28, 2026

Copy link
Copy Markdown
Member

The Categories section listed the keys but never explained how a recipe ends up in a category, which made the fix for a customer publishing under uk.co.acme hard to self-discover.

Adds two subsections: how a recipe name maps to a catalog path (segments matched against the full partial package, unmatched segments synthesized as capitalized names), and what root: true really does — the category is not rendered and its children are hoisted, so a multi-segment reverse DNS prefix needs a root per level (uk and uk.co), with a worked uk.co.acme example. Also corrects the root row in the key table, which claimed the opposite of CategoryTree.getCategories(), and notes that a segment like no must be quoted.

Document how a recipe name maps to a catalog path, what root: true
actually does, and that multi-segment reverse DNS prefixes need a root
category per level.

@mike-solomon mike-solomon 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.

Thank you!

@github-project-automation github-project-automation Bot moved this from In Progress to Ready to Review in OpenRewrite Aug 28, 2026
@timtebeek
timtebeek merged commit 8ae483e into master Aug 28, 2026
2 checks passed
@timtebeek
timtebeek deleted the tim/root-category-docs branch August 28, 2026 15:18
@github-project-automation github-project-automation Bot moved this from Ready to Review to Done in OpenRewrite Aug 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants