Skip to content

Change 'server' to 'service' for consistency#1005

Merged
leebyron merged 1 commit intomainfrom
server-service
Jan 12, 2023
Merged

Change 'server' to 'service' for consistency#1005
leebyron merged 1 commit intomainfrom
server-service

Conversation

@benjie
Copy link
Copy Markdown
Member

@benjie benjie commented Jan 12, 2023

Outside of the introduction/overview, this is the only mention of the word "server" in the entire GraphQL spec. Since the paragraph opens talking about GraphQL services, let's use the term service here for consistency.

@benjie benjie added the ✏️ Editorial PR is non-normative or does not influence implementation label Jan 12, 2023
@netlify
Copy link
Copy Markdown

netlify Bot commented Jan 12, 2023

Deploy Preview for graphql-spec-draft ready!

Name Link
🔨 Latest commit 0acbdea
🔍 Latest deploy log https://app.netlify.com/sites/graphql-spec-draft/deploys/63c00f989c8d7d0008cc6038
😎 Deploy Preview https://deploy-preview-1005--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 settings.

@benjie benjie requested a review from a team January 12, 2023 13:48
Copy link
Copy Markdown
Contributor

@andimarek andimarek left a comment

Choose a reason for hiding this comment

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

I think there are some terms that are acceptable (e.g. backend), but I think the most modern and adequate term is actually service. So very happy to use it really everywhere and avoid confusion.

@leebyron
Copy link
Copy Markdown
Collaborator

Agreed, this was the intent thanks for catching this

@leebyron leebyron merged commit 559063c into main Jan 12, 2023
@leebyron leebyron deleted the server-service branch January 12, 2023 22:18
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