-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
October2021 Edition & Changelog #871
Conversation
32ef0ab
to
fd8f485
Compare
fd8f485
to
dd14865
Compare
changelogs/May2021.md
Outdated
[Brian Warner]: https://github.com/brianwarner | ||
[Chris Brown]: https://github.com/ccbrown | ||
[David Glasser]: https://github.com/glasser | ||
[dugenkui]: https://github.com/graphql/dugenkui03 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[dugenkui]: https://github.com/graphql/dugenkui03 | |
[dugenkui]: https://github.com/dugenkui03 |
changelogs/May2021.md
Outdated
+[Joint Development Foundation](https://www.jointdevelopment.org/). The current | ||
+[Working Group](https://github.com/graphql/graphql-wg) charter, which includes | ||
+the IP policy governing all working group deliverables (including specifications, | ||
+source code, and datasets) may be found at | ||
+[https://technical-charter.graphql.org](https://technical-charter.graphql.org). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sentence should make sense with parenthesis removed, but a missing comma means this doesn't fully make sense:
The current Working Group charter, which includes the IP policy governing all working group deliverables may be found at https://technical-charter.graphql.org.
+[Joint Development Foundation](https://www.jointdevelopment.org/). The current | |
+[Working Group](https://github.com/graphql/graphql-wg) charter, which includes | |
+the IP policy governing all working group deliverables (including specifications, | |
+source code, and datasets) may be found at | |
+[https://technical-charter.graphql.org](https://technical-charter.graphql.org). | |
+[Joint Development Foundation](https://www.jointdevelopment.org/). The current | |
+[Working Group](https://github.com/graphql/graphql-wg) charter, which includes | |
+the IP policy governing all working group deliverables (including specifications, | |
+source code, and datasets), may be found at | |
+[https://technical-charter.graphql.org](https://technical-charter.graphql.org). |
dd14865
to
de24777
Compare
changelogs/May2021.md
Outdated
with the algorithms to power them. The goal of this specification is to provide | ||
a foundation and framework for an ecosystem of GraphQL tools, client libraries, | ||
-and server implementations -- spanning both organizations and platforms -- that | ||
+and service implementations—spanning both organizations and platforms—that |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the new version of this line was hard to read, could we omit "spanning both organizations and platforms"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me. Sorry for my general all around silence. It looks like much of what there is to be found has been addressed already.
de24777
to
cf8a1c8
Compare
✔️ Deploy Preview for graphql-spec-draft ready! 🔨 Explore the source changes: cf8a1c8 🔍 Inspect the deploy log: https://app.netlify.com/sites/graphql-spec-draft/deploys/61784d58ad9e3b0007136def 😎 Browse the preview: https://deploy-preview-871--graphql-spec-draft.netlify.app |
Whoops, sorry for the inadvertent close. Trying to incorporate final feedback before merging this, and messed up the branch 🤦. I'll get this fixed |
@graphql/tsc, please find here the changelog and release candidate for the May2021 edition of the spec:
With the changelog available, we'll open the review and voting process. Once approved, this PR will be merged and the next edition will be live.
The text is not perfect and there are ongoing editorial changes as well as RFCs to further improve it in future editions. If when reviewing you spot something that could be improved, please feel free to drop a review comment on the "diff" section of the changelog within this PR. I'll assume all review comments are "forward looking" to describe future improvements. If you have a concern that will block you from voting to approve this edition please say so explicitly.
Closes #855