FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Backport 25.1: parser cleanup by andimarek · Pull Request #4463 · graphql-java/graphql-java · GitHub

Backport 25.1: parser cleanup - #4463

Merged
andimarek merged 2 commits into
25.xfrom
codex/backport-4442-to-25.x
Aug 24, 2026
Merged

Backport 25.1: parser cleanup#4463
andimarek merged 2 commits into
25.xfrom
codex/backport-4442-to-25.x

Conversation

Copy link
Copy Markdown
Member

Backport of #4442 to the 25.x maintenance branch.

The original commit applies without changes.

Summary:

  • makes block string indentation removal linear by retaining an index window over the split lines
  • avoids repeated removal from the front of an array-backed list
  • adds focused coverage for large runs of leading and trailing blank lines

Testing:

  • RELEASE_VERSION=local-test ./gradlew test -x testWithJava11 -x testWithJava17 --tests graphql.parser.StringValueParsingTest (15 tests on Java 21)

Copy link
Copy Markdown
Contributor

Test Results

  326 files    326 suites   5m 2s ⏱️
5 220 tests 5 211 ✅ 9 💤 0 ❌
5 309 runs  5 300 ✅ 9 💤 0 ❌

Results for commit 7e0726f.

github-actions Bot commented Aug 23, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Test Results

  326 files    326 suites   5m 10s ⏱️
5 220 tests 5 212 ✅ 8 💤 0 ❌
5 309 runs  5 301 ✅ 8 💤 0 ❌

Results for commit 7e0726f.

♻️ This comment has been updated with latest results.

Copy link
Copy Markdown
Contributor

Test Report

Test Results

Java Version Total Passed Failed Errors Skipped
Java 11 - - - - -
Java 17 - - - - -
Java 21 - - - - -
Java 25 - - - - -
jcstress - - - - -

Updated: 2026-08-23 23:58:23 UTC

andimarek merged commit d7a1af7 into 25.x Aug 24, 2026
5 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

1 participant


Back | FazBrowse Home | New Git URL