Skip to content

deps: stick to github.com/vektah/gqlparser/v2@v2.5.30#1744

Merged
anderseknert merged 1 commit intoopen-policy-agent:mainfrom
srenatus:sr/rurylpxryxxy
Nov 3, 2025
Merged

deps: stick to github.com/vektah/gqlparser/v2@v2.5.30#1744
anderseknert merged 1 commit intoopen-policy-agent:mainfrom
srenatus:sr/rurylpxryxxy

Conversation

@srenatus
Copy link
Copy Markdown
Contributor

@srenatus srenatus commented Nov 3, 2025

Because the latest version would require a change to OPA's YAML test cases. For projects that use both Regal and OPA, like EOPA, it's perhaps better to have OPA control the version of third party libs.

The extra field that v2.5.31 returns is ExpectedTypeHasDefault, introduced in vektah/gqlparser@5b41269.

Tl;dr:

Because the latest version would require a change to OPA's YAML
test cases. For projects that use both Regal and OPA, like EOPA,
it's better to have OPA control the version of third party libs.

The extra field that v2.5.31 returns is `ExpectedTypeHasDefault`,
introduced in
vektah/gqlparser@5b41269

Signed-off-by: Stephan Renatus <stephan.renatus@gmail.com>
Copy link
Copy Markdown
Member

@anderseknert anderseknert left a comment

Choose a reason for hiding this comment

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

Yeah, what's messed up here is either the project trying to keep its dependencies up to date, or it's the one introducing breaking changes in patch releases. Hard to say.

@anderseknert anderseknert merged commit 781f809 into open-policy-agent:main Nov 3, 2025
8 checks passed
@srenatus srenatus deleted the sr/rurylpxryxxy branch November 3, 2025 10:59
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.

2 participants