Skip to content

Conversation

@leebyron
Copy link
Collaborator

@leebyron leebyron commented Apr 7, 2021

Factored out of #777. Clarifies when type definition & extension can and cannot be included in executable documents

To do this more clearly, this introduces a number of new grammar rules. This does not change the actual behavior of a parser (all existing docs are parsed equivalently, no new docs are legal). However instead of using language that manipulates existing rules this can just reference those already restricted.

Hopefully this actually is useful for implementations which seek to offer clear subsets for each use case.

@leebyron leebyron added the ✏️ Editorial PR is non-normative or does not influence implementation label Apr 7, 2021
@leebyron leebyron added this to the May2021 milestone Apr 7, 2021
@leebyron leebyron requested review from a team and benjie April 7, 2021 21:33
@leebyron leebyron force-pushed the editorial-document-rules branch 4 times, most recently from bbfa12c to 8429069 Compare April 7, 2021 21:50
Factored out of #777. Clarifies when type definition & extension can and cannot be included in executable documents

To do this more clearly, this introduces a number of new grammar rules. This does not change the actual behavior of a parser (all existing docs are parsed equivalently, no new docs are legal). However instead of using language that manipulates existing rules this can just reference those already restricted.

Hopefully this actually is useful for implementations which seek to offer clear subsets for each use case.
@leebyron leebyron force-pushed the editorial-document-rules branch from 8429069 to ba734b6 Compare April 7, 2021 22:05
@leebyron leebyron merged commit f8b75d3 into main Apr 9, 2021
@leebyron leebyron deleted the editorial-document-rules branch April 9, 2021 04:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✏️ Editorial PR is non-normative or does not influence implementation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants