Skip to content

Support for sh:uniqueValuesFor#808

Open
HolgerKnublauch wants to merge 9 commits intogh-pagesfrom
issue-661
Open

Support for sh:uniqueValuesFor#808
HolgerKnublauch wants to merge 9 commits intogh-pagesfrom
issue-661

Conversation

@HolgerKnublauch
Copy link
Contributor

@HolgerKnublauch HolgerKnublauch commented Mar 8, 2026

@HolgerKnublauch HolgerKnublauch added the Core For SHACL 1.2 Core spec label Mar 8, 2026
@TallTed TallTed changed the title #661 GH#661 - collecting feedback re "Constraint violation on keys" Mar 10, 2026
@HolgerKnublauch HolgerKnublauch marked this pull request as ready for review March 10, 2026 07:46
@HolgerKnublauch HolgerKnublauch changed the title GH#661 - collecting feedback re "Constraint violation on keys" Support for sh:uniqueValuesFor Mar 10, 2026
Copy link
Contributor

@bergos bergos left a comment

Choose a reason for hiding this comment

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

Maybe it's worth mentioning that other constraints define if there are values for the defined properties and how many. Or it could be shown in the example. Since it has some similarities to composite primary keys from SQL, one may expect the values must be sh:minCount == sh:maxCount == 1.

I think that's nice to have for the specification, but it should be covered in a test.

</div>
<p><em>The remainder of this section is informative.</em></p>
<p>
Note that matching of literals needs to be exact, e.g. <code>"04"^^xsd:byte</code> does not match <code>"4"^^xsd:integer</code>.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Note that matching of literals needs to be exact, e.g. <code>"04"^^xsd:byte</code> does not match <code>"4"^^xsd:integer</code>.
Note that matching of literals needs to be exact, e.g. <code>"04"^^xsd:byte</code> does not match <code>"4"^^xsd:integer</code>.
The same applies to language tags, e.g. <code>"Alice"@en</code> does not match <code>"Alice"@fr</code>.

I think it would be a good idea to add a note about language tags, too.
For the sh:uniqueLang the TEXTUAL DEFINITION mentions also base direction.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I had copied that sentence from sh:in, so if we expand on this topic here, we should do this everywhere.

Given that this particular case is just an example ("e.g.") I honestly don't think we need to give another example.

@HolgerKnublauch
Copy link
Contributor Author

@bergos I think I have now addressed your (valid) feedback. Please review again.

@HolgerKnublauch HolgerKnublauch requested a review from bergos March 12, 2026 02:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Core For SHACL 1.2 Core spec

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Constraint violation on keys

3 participants