-
Notifications
You must be signed in to change notification settings - Fork 28
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
refactor: move section "Terminology and conventions" right after "Conformance" #353
Conversation
<p> | ||
<dfn>JSON Schema</dfn> is defined in <a href="https://json-schema.org/specification.html">these specifications</a>. | ||
</p> | ||
<p> |
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 terminology up to this point should be fine. But below this should go to the end of the doc, it's more like an addendum. If we use the proper Web platform citations, I think we can remove the stuff from here on.
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.
More recent specs don't even have this section, as they adhere to the Web Platform and use citations from e.g. https://respec.org/xref/ or bikeshed refs https://tabatkins.github.io/bikeshed/#cli-refs
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.
Note: I linked the tool for ReSpec, see https://github.com/w3c/wot-scripting-api/blob/main/README.md#editors-tools
Scripting Call 2021-11-29
|
I manged to get rid of all terms except ECMASCRIPT... removing causes lots of issues (defined in several places). Moreover, xref does not show proper definitions for "JSON" etc. Note: I also fixed an issue with |
@@ -3940,99 +4037,6 @@ <h3>Denial Of Service Security Risk</h3> | |||
</section> | |||
</section> | |||
|
|||
<section> <h2>Terminology and conventions</h2> |
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.
Instead of removing, we could just put these in a comment?
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.
It has not been removed. It was moved to the top as requested.
Scripting Call 2021-12-06: |
resolves part#1 of #342
Preview | Diff