Skip to content

Conversation

@martinbonnin
Copy link
Contributor

@netlify
Copy link

netlify bot commented Feb 25, 2025

Deploy Preview for graphql-spec-draft ready!

Name Link
🔨 Latest commit 540392c
🔍 Latest deploy log https://app.netlify.com/sites/graphql-spec-draft/deploys/67bdc296cb6a3f00086f795f
😎 Deploy Preview https://deploy-preview-1137--graphql-spec-draft.netlify.app
📱 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.

Copy link
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.

@martinbonnin Please split this into two PRs, the definitions can easily be merged, but I think we should ask the editor to approve the tone of voice section.

@martinbonnin martinbonnin mentioned this pull request Feb 25, 2025
@martinbonnin
Copy link
Contributor Author

@benjie done. Follow up PR for tone of voice is at #1138

Copy link
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.

They also automatically link all italicised references to the term.

@martinbonnin martinbonnin changed the title Add tone of voice and definitions to the style guide Add definitions to the style guide Feb 25, 2025
@benjie benjie merged commit 60a0677 into graphql:main Feb 26, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants