Skip to content

Fix doubled whitespace for inline fragments #355

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 4 commits into from
Oct 13, 2023
Merged

Fix doubled whitespace for inline fragments #355

merged 4 commits into from
Oct 13, 2023

Conversation

sungam3r
Copy link
Member

fixes #352

@sungam3r sungam3r added the bugfix Pull request that fixes a bug label Oct 13, 2023
@sungam3r sungam3r requested a review from Shane32 October 13, 2023 05:07
@sungam3r sungam3r self-assigned this Oct 13, 2023
@github-actions github-actions bot added the test Pull request that adds new or changes existing tests label Oct 13, 2023
@sungam3r
Copy link
Member Author

@Shane32 See https://github.com/graphql-dotnet/parser/actions/runs/6503896922 . It seems that jumping with porting commits across branches may break something else.

@sungam3r
Copy link
Member Author

I copied codeql-analysis.yml from master into v8.

@sungam3r sungam3r merged commit bd24417 into v8 Oct 13, 2023
@sungam3r sungam3r deleted the whitespaces branch October 13, 2023 05:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Shane32 Shane32 Shane32 approved these changes

Assignees

@sungam3r sungam3r

Labels
bugfix Pull request that fixes a bug test Pull request that adds new or changes existing tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

There's an extra space here also, between ... and @skip.
2 participants
@sungam3r @Shane32