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

Update links in SPARQL update conformance for syntax #52

Merged
merged 1 commit into from
Feb 6, 2025

Conversation

afs
Copy link
Contributor

@afs afs commented Jan 29, 2025

@afs afs force-pushed the syntax-conformance branch from 1f98573 to 6145514 Compare January 29, 2025 17:56
@afs afs force-pushed the syntax-conformance branch from 6145514 to af757d4 Compare January 29, 2025 18:00
Comment on lines +1824 to +1827
<p>See
<a data-cite="SPARQL12-QUERY#dfn-sparql-update-string">SPARQL update string</a>
and the <a data-cite="SPARQL12-QUERY#grammar">SPARQL grammar</a>
regarding the SPARQL update language.
Copy link
Member

Choose a reason for hiding this comment

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

Capitalization taken from the linked document. "Update Grammar" is only capitalized there when it's in a title; "Update String" and "Update Language" are capitalized within prose blocks.

Suggested change
<p>See
<a data-cite="SPARQL12-QUERY#dfn-sparql-update-string">SPARQL update string</a>
and the <a data-cite="SPARQL12-QUERY#grammar">SPARQL grammar</a>
regarding the SPARQL update language.
<p>See
<a data-cite="SPARQL12-QUERY#dfn-sparql-update-string">SPARQL Update String</a>
and the <a data-cite="SPARQL12-QUERY#grammar">SPARQL grammar</a>
regarding the SPARQL Update Language.

Copy link
Contributor Author

@afs afs Jan 30, 2025

Choose a reason for hiding this comment

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

The PR on the other document makes it "SPARQL update string" because, elsewhere, it is "RDF string". This is making things more consistent across all our documents.

"SPARQL update language" is not a formal, defined term.
In titles, it might be "SPARQL Update Language".

Copy link
Member

Choose a reason for hiding this comment

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

I think SPARQL Update string is a more sensible parallel to RDF string, because This document describes SPARQL 1.2 Update, an update language for RDF graphs.

(I do find SPARQL 1.2 Update, etc., to be clumsier than SPARQL Update 1.2, etc., and it may be worth revisiting these titles.)

Copy link
Contributor Author

@afs afs Jan 30, 2025

Choose a reason for hiding this comment

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

Upper-lower in the terminology looks odd. It's one phrase.

For the change of document title, you'll need to take that to the chairs. It's in the charter.

Copy link
Member

Choose a reason for hiding this comment

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

Upper-lower in the terminology looks odd. It's one phrase.

SPARQL update (or SPARQL 1.2 update) doesn't exist.
SPARQL Update or SPARQL 1.2 Update does.

Copy link
Contributor Author

@afs afs Feb 6, 2025

Choose a reason for hiding this comment

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

See w3c/sparql-query#190.

This PR aligns to the grammar terminology in the query document.

@afs afs merged commit 1d1dcfb into main Feb 6, 2025
2 checks passed
@afs afs deleted the syntax-conformance branch February 6, 2025 11:19
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