Skip to content

fix(json): preserve sentinel hierarchy (#59) - #80

Open
PandaHUN777 wants to merge 1 commit into
go-extras:masterfrom
PandaHUN777:fix/serialize-sentinel-ancestors-59
Open

PandaHUN777 wants to merge 1 commit into
go-extras:masterfrom
PandaHUN777:fix/serialize-sentinel-ancestors-59

Conversation

@PandaHUN777

Copy link
Copy Markdown
Contributor

Add a sentinel_ancestors JSON field for pure sentinel chains while keeping the existing sentinels field unchanged. Closes #59. Validation: go test ./..., go test -race -covermode=atomic -coverprofile=/tmp/errx-59-coverage.out ./..., go build ./..., go vet ./..., golangci-lint run, govulncheck ./..., git diff --check.

@coderabbitai

coderabbitai Bot commented Sep 30, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

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

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: af979268-4d4c-408d-affe-920c938df4bf

  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


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.

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.

json: sentinel hierarchy is lost in serialized output

1 participant