Skip to content

fix: resolve $ref/$inline/$merge on root components - #21

Open
Macawls wants to merge 1 commit into
dolmen-go:masterfrom
Macawls:fix/components-with-directive
Open

Macawls wants to merge 1 commit into
dolmen-go:masterfrom
Macawls:fix/components-with-directive

Conversation

@Macawls

@Macawls Macawls commented Oct 11, 2026

Copy link
Copy Markdown

Summary

  • Fix: root-level components holding a \$ref, \$inline or \$merge directive is now expanded instead of being silently skipped
  • Add hasDirective helper to detect directive keys on a map value
  • Add testdata case 47-inline-components covering \$inline on root components

Closes #9

Test plan

  • gofmt -l refs.go util.go
  • go vet ./...
  • go build ./...
  • go test ./...

When the root-level "components" key holds a $ref, $inline or
$merge directive, it is now expanded instead of being skipped.

Closes dolmen-go#9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

YAML objects named "components" don't resolve

1 participant