-
Notifications
You must be signed in to change notification settings - Fork 525
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
Add WARNING for camelized identifiers #208
Comments
This seems related to #185. |
I believe yes |
Just to start the discussion, keep in mind that this is a bit hard at the moment because adapters are a runtime phenomena. That is at compile time when the schema is built, we have no idea what convention you're gonna use. |
Because regardless of which runtime adapter is being used, the internal representation will stay static (ie, the schema convention can't change), and because Slated for v1.3's schema definition changes. |
@bruce do we want to go ahead with making non snake case field identifiers an error or warning? |
Note that it will likely help with stuff like #298 |
I think we should just error. |
As a schema writer I want a big WARNING when I write a camelized identifier, i.e.:
field :someField
in order to avoid possible errors when querying/mutating the schema.The text was updated successfully, but these errors were encountered: