Skip to content

consistent indentation and punctuation#925

Merged
leebyron merged 1 commit intographql:mainfrom
thomasheyenbrock:editorial/validation-punctuation-indentation
Jun 2, 2022
Merged

consistent indentation and punctuation#925
leebyron merged 1 commit intographql:mainfrom
thomasheyenbrock:editorial/validation-punctuation-indentation

Conversation

@thomasheyenbrock
Copy link
Copy Markdown
Contributor

@thomasheyenbrock thomasheyenbrock commented Jan 24, 2022

Hello dear spec-maintainers 👋 I noticed some inconsistencies in terms of indentation and punctuation in the validation section, tried to clean it up and make it all consistent.

I also did a tiny change to the spec for Single root field, where I lifted up one declaration that is not depending on the context of the loop.

@netlify
Copy link
Copy Markdown

netlify Bot commented Jan 24, 2022

Copy link
Copy Markdown
Member

@benjie benjie left a comment

Choose a reason for hiding this comment

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

Thanks for doing this, I had meant to do it after the prettier PR (#727) was merged since I discovered a lot of these issues whilst working on that, but alas it took a while and completely slipped off my radar!

Comment thread spec/Section 5 -- Validation.md
@benjie benjie added the ✏️ Editorial PR is non-normative or does not influence implementation label Feb 3, 2022
@leebyron leebyron merged commit a61e35d into graphql:main Jun 2, 2022
@thomasheyenbrock thomasheyenbrock deleted the editorial/validation-punctuation-indentation branch June 11, 2022 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✏️ Editorial PR is non-normative or does not influence implementation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants