Skip to content

Conversation

@martinbonnin
Copy link
Contributor

Follow up from https://github.com/graphql/graphql-scalars/pull/31/files, Custom and Scalar are not capitalized in the spec. String is only capitalized when it is the name of the built-in String scalar. But not in the context of a "string" value.

@martinbonnin
Copy link
Contributor Author

Thanks for the review @glen-84, this all makes sense to me 👍

@martinbonnin
Copy link
Contributor Author

martinbonnin commented Dec 1, 2025

BTW, Title Case is mentioned in the spec style guide there: https://github.com/graphql/graphql-spec/blob/main/STYLE_GUIDE.md#title-case

@martinbonnin
Copy link
Contributor Author

@andimarek ok to merge this one?

@martinbonnin
Copy link
Contributor Author

Since this is an easy PR to revert, I'll go ahead an merge it. @andimarek let me know if you have any objection.

@martinbonnin martinbonnin merged commit 8b3cead into main Dec 17, 2025
5 checks passed
@martinbonnin martinbonnin deleted the decapitalize branch December 17, 2025 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants