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

Fix heading level for Required Arguments validation rule #1055

Closed goto-bus-stop closed 1 year ago

goto-bus-stop commented 1 year ago

Currently in the rendered spec, Required Arguments shows up as a sub-rule of "Argument Uniqueness". It's an independent rule so it should show up next to it instead.

image
netlify[bot] commented 1 year ago

Deploy Preview for graphql-spec-draft ready!

Name Link
Latest commit 901c4a998d593ba503ddbcfa8c061707a869053f
Latest deploy log https://app.netlify.com/sites/graphql-spec-draft/deploys/654cd6c485bd3d0008711eab
Deploy Preview https://deploy-preview-1055--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.

linux-foundation-easycla[bot] commented 1 year ago

CLA Signed

The committers listed above are authorized under a signed CLA.

benjie commented 1 year ago

(You need to sign the CLA before we can merge this fix.)

goto-bus-stop commented 1 year ago

Done! 😁

benjie commented 1 year ago

Congrats on your first merge; and thanks for the fix! :raised_hands: