-
Notifications
You must be signed in to change notification settings - Fork 33
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
(closes #555) Fall back to default logic in useNativeTypes
mode for RDF numbers which are not JSON numbers
#625
base: main
Are you sure you want to change the base?
(closes #555) Fall back to default logic in useNativeTypes
mode for RDF numbers which are not JSON numbers
#625
Conversation
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.
Minor changes, but this direction seems like the way to go.
…ted to JSON number, fall back to default logic
…iveTypes` mode
fd7684b
to
3b61a1b
Compare
|
This was discussed during the #json-ld meeting on 15 January 2025. View the transcriptw3c/json-ld-api#625anatoly-scherbakov: this is the new version of w3c/json-ld-api#619 ; thanks gkellogg and pchampin for your suggestions. gkellogg: it's a fairly small change. anatoly-scherbakov: I can not wrap a whole block in <ins> or <del>, right? bigbluehat: correct, they are 'inline'. Is there a specific class to use? gkellogg: there are other things to put in place. Respec documentation for them is not great, but there are examples in the same doc. anatoly-scherbakov: what is the use-case for this marking? gkellogg: it is there for reviewers of the specification, because we are editing a published Recommendation. anatoly-scherbakov: an HTML diff will not be enough for the reviewers? gkellogg: no, they do not expect to look at HTML raw code. <gkellogg> https://www.w3.org/policies/process/#candidate-amendments bigbluehat: this is required until we recharter. The alternative is to keep a bunch of open PR and merge them only after we recharter. |
This is my second attempt to fix this issue after #619.
Preview | Diff