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

Recognize rdf:dirLangString #60

Closed
wants to merge 1 commit into from
Closed

Recognize rdf:dirLangString #60

wants to merge 1 commit into from

Conversation

afs
Copy link

@afs afs commented Dec 12, 2024

See #59.

PR is included in #64.


Preview | Diff

spec/index.html Outdated
@@ -815,8 +815,9 @@ <h2>Literals and datatypes</h2>
for that datatype.
</p>
<p>RDF processors are not required to <a>recognize</a> any datatype IRIs other than
<a data-cite="RDF12-CONCEPTS#dfn-language-tagged-string"><code>rdf:langString</code></a>
and <a data-cite="XMLSCHEMA11-2#string"><code>xsd:string</code></a>,
<a data-cite="XMLSCHEMA11-2#string"><code>xsd:string</code></a>,
Copy link
Author

Choose a reason for hiding this comment

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

Reordered to put the more common datatype first.

Copy link
Contributor

@pfps pfps left a comment

Choose a reason for hiding this comment

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

Needs a discussion of whether Concepts needs to be changed.

pfps

This comment was marked as outdated.

@pfps pfps added the spec:enhancement Change to enhance the spec without affecting conformance (class 2) –see also spec:editorial label Dec 12, 2024
@afs
Copy link
Author

afs commented Dec 12, 2024

Needs a discussion of whether Concepts needs to be changed.

What change might be necessary?

@pfps
Copy link
Contributor

pfps commented Dec 13, 2024

Concepts doesn't say that any datatype is REQUIRED, which seems to me to be needed if Semantics says that RDF reasoning REQUIRES some datatypes.

@afs
Copy link
Author

afs commented Dec 13, 2024

This PR is not about the general issue.

RDF Semantics does require xsd:string, rdf:langString, therefore it should require rdf:dirLangString. This should have been done when the WG completed changes for base direction. It got missed and this PR ensures it does not get lost.

The wider issue is elsewhere:
w3c/rdf-concepts#60 (comment)

@pfps
Copy link
Contributor

pfps commented Dec 13, 2024

I created w3c/rdf-concepts#122 explicitly about whether Concepts should discuss requirements for RDF entailment.

@pchampin
Copy link
Contributor

@afs after reverting #52 per our resolution today, I tried to rebased this PR, but the formatting change created a conflict :-/ . I will work on it a little later.

@pchampin
Copy link
Contributor

@afs after reverting #52 per our resolution today, I tried to rebased this PR, but the formatting change created a conflict :-/ . I will work on it a little later.

that's now fixed. This PR is rebased on the new main (i.e. after #52 has been reverted)

@gkellogg
Copy link
Member

@afs after reverting #52 per our resolution today, I tried to rebased this PR, but the formatting change created a conflict :-/ . I will work on it a little later.

If there are no intervening commits to preserve, it might be easier to just reset the head and force-push to eliminate those commits.

@afs
Copy link
Author

afs commented Dec 20, 2024

Rebased to current main, preview and diff rebuilt.

Editors - this is ready to merge.

@afs
Copy link
Author

afs commented Dec 21, 2024

Closing. This PR is included in #64.
There are more areas that need updating for rdf:dirLangString.

@afs afs closed this Dec 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec:enhancement Change to enhance the spec without affecting conformance (class 2) –see also spec:editorial
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants