Skip to content
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

Automated Section renumber and grammar extraction #504

Merged
merged 1 commit into from
Mar 17, 2022

Conversation

github-actions[bot]
Copy link
Contributor

renumber sections. Add grammar

@github-actions github-actions bot force-pushed the create-pull-request/patch branch from 1d483d0 to 6d3dbbb Compare March 16, 2022 19:28
@@ -370,7 +370,7 @@
- [§11.17.2](expressions.md#11172-ambiguities-in-query-expressions) Ambiguities in query expressions
- [§11.17.3](expressions.md#11173-query-expression-translation) Query expression translation
- [§11.17.3.1](expressions.md#111731-general) General
- [§11.17.3.2](expressions.md#111732-select-and-group--by-clauses-with-continuations) select and group ... by clauses with continuations
- [§11.17.3.2](expressions.md#111732-select-and-group--by-clauses-with-continuations) select and group by clauses with continuations
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's going to be interesting... I've no idea how browsers will handle URLs with an ellipsis in. I don't mind trying it though...

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This might require a change in the tool. The ellipsis probably isn't in the HTML anchor. After this is merged, I'll check.

@BillWagner BillWagner merged commit a40b509 into draft-v6 Mar 17, 2022
@BillWagner BillWagner deleted the create-pull-request/patch branch March 17, 2022 13:43
Copy link
Contributor

@Nigel-Ecma Nigel-Ecma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Found one error due to missing space, raised #523 to fix it. Otherwise LGTM

BillWagner pushed a commit that referenced this pull request Mar 18, 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.

4 participants