Skip to content

upgrade antlr #3997

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Jun 2, 2025
Merged

upgrade antlr #3997

merged 1 commit into from
Jun 2, 2025

Conversation

andimarek
Copy link
Member

No description provided.

@andimarek andimarek requested a review from Copilot June 2, 2025 11:11
Copilot
Copilot AI reviewed Jun 2, 2025
View reviewed changes
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

@andimarek andimarek added the not release related changes which are not released (for example unit tests or docs) label Jun 2, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jun 2, 2025

Test Results

  318 files  ±0    318 suites  ±0   2m 35s ⏱️ +4s
4 677 tests ±0  4 668 ✅ ±0  9 💤 ±0  0 ❌ ±0 
4 766 runs  ±0  4 757 ✅ ±0  9 💤 ±0  0 ❌ ±0 

Results for commit c7495f9. ± Comparison against base commit 19387ff.

This pull request removes 192 and adds 170 tests. Note that renamed tests count towards both. 
	?
                __schema { types { fields { args { type { name fields { name }}}}}}
                __schema { types { fields { type { name fields { name }}}}}
                __schema { types { inputFields { type { inputFields { name }}}}}
                __schema { types { interfaces { fields { type { interfaces { name } } } } } }
                __schema { types { name} }
                __type(name : "t") { name }
                a1: __schema { types { name} }
                a1: __type(name : "t") { name }
                a2 :  __type(name : "t1") { name }
…

graphql.AssertTest ‑ assertFalse with different number of error args but false does not throw assertions [toRun: <graphql.AssertTest$__spock_feature_0_21prov0_closure23@748e9b20 delegate=graphql.AssertTest@6edcd0d8 owner=graphql.AssertTest@6edcd0d8 thisObject=graphql.AssertTest@6edcd0d8 resolveStrategy=0 directive=0 parameterTypes=[class java.lang.Object] maximumNumberOfParameters=1 bcw=null>, expectedMessage: error arg1, #0]
graphql.AssertTest ‑ assertFalse with different number of error args but false does not throw assertions [toRun: <graphql.AssertTest$__spock_feature_0_21prov0_closure24@50f40653 delegate=graphql.AssertTest@6edcd0d8 owner=graphql.AssertTest@6edcd0d8 thisObject=graphql.AssertTest@6edcd0d8 resolveStrategy=0 directive=0 parameterTypes=[class java.lang.Object] maximumNumberOfParameters=1 bcw=null>, expectedMessage: error arg1 arg2, #1]
graphql.AssertTest ‑ assertFalse with different number of error args but false does not throw assertions [toRun: <graphql.AssertTest$__spock_feature_0_21prov0_closure25@50a3d0f6 delegate=graphql.AssertTest@6edcd0d8 owner=graphql.AssertTest@6edcd0d8 thisObject=graphql.AssertTest@6edcd0d8 resolveStrategy=0 directive=0 parameterTypes=[class java.lang.Object] maximumNumberOfParameters=1 bcw=null>, expectedMessage: error arg1 arg2 arg3, #2]
graphql.AssertTest ‑ assertFalse with different number of error args throws assertions [toRun: <graphql.AssertTest$__spock_feature_0_20prov0_closure20@1f966492 delegate=graphql.AssertTest@6edcd0d8 owner=graphql.AssertTest@6edcd0d8 thisObject=graphql.AssertTest@6edcd0d8 resolveStrategy=0 directive=0 parameterTypes=[class java.lang.Object] maximumNumberOfParameters=1 bcw=null>, expectedMessage: error arg1, #0]
graphql.AssertTest ‑ assertFalse with different number of error args throws assertions [toRun: <graphql.AssertTest$__spock_feature_0_20prov0_closure21@63fdffcd delegate=graphql.AssertTest@6edcd0d8 owner=graphql.AssertTest@6edcd0d8 thisObject=graphql.AssertTest@6edcd0d8 resolveStrategy=0 directive=0 parameterTypes=[class java.lang.Object] maximumNumberOfParameters=1 bcw=null>, expectedMessage: error arg1 arg2, #1]
graphql.AssertTest ‑ assertFalse with different number of error args throws assertions [toRun: <graphql.AssertTest$__spock_feature_0_20prov0_closure22@7428de63 delegate=graphql.AssertTest@6edcd0d8 owner=graphql.AssertTest@6edcd0d8 thisObject=graphql.AssertTest@6edcd0d8 resolveStrategy=0 directive=0 parameterTypes=[class java.lang.Object] maximumNumberOfParameters=1 bcw=null>, expectedMessage: error arg1 arg2 arg3, #2]
graphql.AssertTest ‑ assertNotNull with different number of  error args throws assertions [toRun: <graphql.AssertTest$__spock_feature_0_5prov0_closure3@4c2869a9 delegate=graphql.AssertTest@6edcd0d8 owner=graphql.AssertTest@6edcd0d8 thisObject=graphql.AssertTest@6edcd0d8 resolveStrategy=0 directive=0 parameterTypes=[class java.lang.Object] maximumNumberOfParameters=1 bcw=null>, expectedMessage: error arg1, #0]
graphql.AssertTest ‑ assertNotNull with different number of  error args throws assertions [toRun: <graphql.AssertTest$__spock_feature_0_5prov0_closure4@20095ab4 delegate=graphql.AssertTest@6edcd0d8 owner=graphql.AssertTest@6edcd0d8 thisObject=graphql.AssertTest@6edcd0d8 resolveStrategy=0 directive=0 parameterTypes=[class java.lang.Object] maximumNumberOfParameters=1 bcw=null>, expectedMessage: error arg1 arg2, #1]
graphql.AssertTest ‑ assertNotNull with different number of  error args throws assertions [toRun: <graphql.AssertTest$__spock_feature_0_5prov0_closure5@35ff8fc9 delegate=graphql.AssertTest@6edcd0d8 owner=graphql.AssertTest@6edcd0d8 thisObject=graphql.AssertTest@6edcd0d8 resolveStrategy=0 directive=0 parameterTypes=[class java.lang.Object] maximumNumberOfParameters=1 bcw=null>, expectedMessage: error arg1 arg2 arg3, #2]
graphql.AssertTest ‑ assertNotNull with different number of error args with non null does not throw assertions [toRun: <graphql.AssertTest$__spock_feature_0_6prov0_closure6@9a2ec9b delegate=graphql.AssertTest@6edcd0d8 owner=graphql.AssertTest@6edcd0d8 thisObject=graphql.AssertTest@6edcd0d8 resolveStrategy=0 directive=0 parameterTypes=[class java.lang.Object] maximumNumberOfParameters=1 bcw=null>, expectedMessage: error arg1, #0]
…

@andimarek andimarek merged commit 589b33d into master Jun 2, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

Copilot code review Copilot Copilot left review comments

Assignees
No one assigned
Labels
not release related changes which are not released (for example unit tests or docs)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@andimarek