search

graphql / graphql-spec

GraphQL is a query language and execution engine tied to any backend service.
https://spec.graphql.org
14.31k stars 1.13k forks source link

Enforce consistent punctuation in algorithms #1069

Closed benjie closed 8 months ago

benjie commented 11 months ago

Follow up to #1067 with thorough check through all algorithm steps, and a CI check to ensure there's no regressions.

Using consistent formatting within the spec will make it easier to read, easier to search through, and easier for automated tools to parse it.

cc @yaacovCR

netlify[bot] commented 11 months ago

Deploy Preview for graphql-spec-draft ready!

Name Link
Latest commit a303f0c62f4a76782cacded7f8ddcff90455b8c6
Latest deploy log https://app.netlify.com/sites/graphql-spec-draft/deploys/657b2b473edb2600085bf199
Deploy Preview https://deploy-preview-1069--graphql-spec-draft.netlify.app/draft
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

benjie commented 11 months ago

@graphql/tsc Can I get a couple +1's on this? You can skip over the code of the formatting script if you want, that's just a script to enforce the changes. The actual spec changes are mostly adding . or : and wrapping true/false/null in {}.