Skip to content

GraphQL: Fix missing space after keyword in anonymous operations#10689

Merged
thorn0 merged 2 commits intoprettier:mainfrom
patriscus:issue10655
Apr 12, 2021
Merged

GraphQL: Fix missing space after keyword in anonymous operations#10689
thorn0 merged 2 commits intoprettier:mainfrom
patriscus:issue10655

Conversation

@patriscus
Copy link
Copy Markdown
Contributor

Fixes #10655

Description

Added an additional condition to check if a space (" ") should be inserted after the keyword.

Checklist

  • I’ve added tests to confirm my change works.
  • (If changing the API or CLI) I’ve documented the changes I’ve made (in the docs/ directory).
  • (If the change is user-facing) I’ve added my changes to changelog_unreleased/*/XXXX.md file following changelog_unreleased/TEMPLATE.md.
  • I’ve read the contributing guidelines.

Try the playground for this PR

@patriscus patriscus changed the title Issue10655 GraphQL: Fix missing space after keyword in anonymous operations (#10655) Apr 11, 2021
GraphQL: Fix missing space after keyword in anonymous operations
Copy link
Copy Markdown
Member

@thorn0 thorn0 left a comment

Choose a reason for hiding this comment

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

Looks good. Thank you for your contribution!

Comment thread changelog_unreleased/graphql/10655.md Outdated
@@ -0,0 +1,19 @@
#### Fix missing space after keyword in anonymous graphql operations (#10655 by @patriscus)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could you please rename the file and specify the number of the PR here instead of the number of the issue?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@thorn0 thanks for reviewing so promptly and sorry about the wrong number, changed it now

Add changelog
@patriscus patriscus changed the title GraphQL: Fix missing space after keyword in anonymous operations (#10655) GraphQL: Fix missing space after keyword in anonymous operations (#10689) Apr 11, 2021
@brody2consult
Copy link
Copy Markdown

Is this expected to make it into the 2.3 release?

@patriscus I think there is no need to put the PR number in the title of this PR. The GitHub UI will generally add the PR number when the PR is merged.

@patriscus patriscus changed the title GraphQL: Fix missing space after keyword in anonymous operations (#10689) GraphQL: Fix missing space after keyword in anonymous operations Apr 11, 2021
@thorn0 thorn0 merged commit cbdc8aa into prettier:main Apr 12, 2021
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Apr 13, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GraphQL anonymous operations have incorrect spacing after keyword

4 participants