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

modify make token by byte position, not rune position at readName #709

Closed

saranrapjs wants to merge 1 commit into graphql-go:master from nytimes:fix/multi-byte-parse

saranrapjs commented Jan 3, 2025

Incorporating the patch from this fork into our fork, which fixes a lexing bug with multi-byte comment sequences: #605


          modify make token by byte position, not rune position at readName

ea0716e

coveralls commented Jan 3, 2025

coverage: 92.422% (+0.4%) from 92.048%
when pulling ea0716e on nytimes:fix/multi-byte-parse
into a546af7 on graphql-go:master.

saranrapjs closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet