test(go): stabilize RawMessage contract type on Go 1.27 by BLCCoreStudio · Pull Request #1737 · grafana/grafana-pathfinder-app · GitHub
Skip to content

test(go): stabilize RawMessage contract type on Go 1.27 - #1737

Merged
moxious merged 1 commit into
grafana:mainfrom
BLCCoreStudio:fix/go127-rawmessage-contract
Aug 29, 2026
Merged

test(go): stabilize RawMessage contract type on Go 1.27#1737
moxious merged 1 commit into
grafana:mainfrom
BLCCoreStudio:fix/go127-rawmessage-contract

Conversation

@BLCCoreStudio

Copy link
Copy Markdown
Contributor

Summary

Go 1.27 reflects json.RawMessage as jsontext.Value, which makes the contract golden test fail.

Handle json.RawMessage before the generic named-type case. Added cases for json.RawMessage and []json.RawMessage to keep the descriptor consistent on Go 1.26.5 and Go 1.27.

How to verify

  1. Run the focused contract tests with Go 1.26.5 and Go 1.27.
  2. Confirm the existing struct-tag golden stays unchanged.

Breaking changes

None.

Fixes #1724

@cla-assistant

cla-assistant Bot commented Aug 29, 2026

Copy link
Copy Markdown

@cla-assistant

cla-assistant Bot commented Aug 29, 2026

Copy link
Copy Markdown

@BLCCoreStudio
BLCCoreStudio marked this pull request as ready for review August 29, 2026 19:43

@moxious moxious left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for stabilizing the RawMessage contract descriptor across Go toolchains — this keeps the repository’s contract golden reliable as Go 1.27 lands.

No blocking issues. This PR is mergeable.

PR Review: #1737
Summary: stabilize RawMessage contract type on Go 1.27
Verdict: Approve
Results: 0 blockers, 0 non-blocking findings, 0 follow-ups

Once this approval is published, a maintainer can take the normal merge decision.

@moxious
moxious merged commit 243648d into grafana:main Aug 29, 2026
35 of 36 checks passed
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.

test(go): reflected contract golden fails under Go 1.27

2 participants