WorksHub / leona

spec -> lacinia schema
Eclipse Public License 1.0
36 stars 17 forks source link

Update the casing system to respect context #35

Closed acron0 closed 5 years ago

acron0 commented 5 years ago

In this PR we fully revamp the casing system so that it no longer indiscriminately snake_cases everything. Now we follow the GraphQL convention of...

Obviously this has big reverberations and affects a lot of the code base. Almost all the tests needed touching.

The motivation behind the PR is that people on the outside world are exposed to a GraphQL API which is follows the conventions and appears relatively...standard. By snake casing everything we were ignoring the suggested conventions and presented an odd looking API as a result.

The next step is to give Leona users control over exactly which casing is used where. This will appear in a follow-up PR.

acron0 commented 5 years ago

This PR replaces https://github.com/WorksHub/leona/pull/20 and builds on the great work done by @JPRoland