Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix typo in the HCL spec #716

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

amirhosseinab
Copy link

Change schemata to schema

Change `schemata` to `schema`
@amirhosseinab amirhosseinab requested a review from a team as a code owner December 5, 2024 10:30
Copy link

hashicorp-cla-app bot commented Dec 5, 2024

CLA assistant check
All committers have signed the CLA.

@crw
Copy link
Contributor

crw commented Dec 5, 2024

Thanks for your submission! I believe the author was intentionally using schemata as the plural form of schema. This usage appears to be correct if uncommon. I suspect this would not fit in with HashiCorp's style guide today, but it is unclear if this document would be subject to that style guide. Which is a long-winded way of saying, this may or may not be accepted but thanks for pointing it out!

@apparentlymart
Copy link
Contributor

apparentlymart commented Dec 5, 2024

This text originated in the spec from ZCL, which was my personal project that HashiCorp later adopted as HCL 2, so I'm afraid this text is my fault.

@crw is right that this was intended to be the plural of "schema", because it's describing the potentially-many things that can appear inside a singular "body schema". If I'd written this in the "HashiCorp writing voice" then I expect each of the modifications made here would say "schemas" (plural) instead of either "schemata" or "schema", and I'd think that a reasonable change to accept here but of course that's not my call to make. 😀

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.

3 participants