Update spec references. #53
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ing an external description of the diagram. The diagram is now maintained using Google Drawings. Fixes #51
iherman
approved these changes
Aug 22, 2018
There was a problem hiding this comment.
I find it strange what respec generates into the spec for an IDL reference
- the 2016 version is the WebIDL 1.0 spec, and McCormack is its sole editor. If all we use is 1.0, that should work for us: https://www.w3.org/TR/WebIDL-1/
- if we want to refer to the WebIDL 2.0 (I am not sure we can do that on long term, but it is fine in a spec), the URL is indeed https://heycam.github.io/webidl/, then we should call it explicitly webIDL 2.0, but then the date (2016) is incorrect. It is a "living standard", meaning that the date is irrelevant...
|
I tried defining "WebIDL-1" and using it, but ReSpec emits standard references to WebIDL, so can't remove that. But, it sounds like the issue is with SpecRef not having the right definition for WebIDL, and the problem should be fixed there. We could override in our own local bibliography, but that doesn't change what ReSpec emits. |
gkellogg
force-pushed
the
update-references
branch
from
c210fa3 to
11c9583
Compare
|
If we use our own bibliography with a different key, that should work. But it is not urgent. These things become important when we get to a final document... |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #50.
See also w3c/json-ld-api#24 and w3c/json-ld-framing#4.
Preview | Diff