Skip to content

Update CCN RFC, and introduce a decision log #966

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

Merged
merged 7 commits into from
May 16, 2022
Merged

Conversation

twof
Copy link
Contributor

@twof twof commented May 14, 2022

This proposal started out with a very simple premise and implementation, and has gotten more complex as
the community has explored edge cases and facets about how GraphQL is actually used in practice. For
example this proposal starts out by talking about accommodating the "best practices" that are recommended
by the GraphQL documentation and the community, but we discovered pretty early on that there are
legitimate use cases where the "best practices" are rightfully ignored.

In order to cover instances like that, we've needed to justify additional complexity which can be
difficult to understand for newcomers without (at this point a full year of) context. This decision
log was written with newcomers in mind to avoid rediscussing issues that have already been hashed out,
and to make it easier to understand why certain decisions have been made. At the time of writing,
the decisions here aren't set in stone, so any future discussions can use this log as a starting point.

@twof twof changed the title Ccn update Update proposal, and introduce a decision log May 15, 2022
@twof twof changed the title Update proposal, and introduce a decision log Update CCN RFC, and introduce a decision log May 15, 2022
@twof twof marked this pull request as ready for review May 15, 2022 04:01
@benjie benjie merged commit e975ecc into graphql:main May 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants