Skip to content

Issue 163 path#620

Open
VladimirAlexiev wants to merge 190 commits into
w3c:gh-pagesfrom
VladimirAlexiev:issue-163-PATH
Open

Issue 163 path#620
VladimirAlexiev wants to merge 190 commits into
w3c:gh-pagesfrom
VladimirAlexiev:issue-163-PATH

Conversation

@VladimirAlexiev

@VladimirAlexiev VladimirAlexiev commented Oct 23, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

Closes #163

  • See this document rendered online here
  • @HolgerKnublauch Sorry, I made a github mistake and this includes a gazillion commits. Can you pick only the last one 8d3d823 ?
  • I've deleted a few duplicated words from a definition and replaced with a reference to the section that defines "substitution" fully.
  • Added suggestion to emit "BAD-PATH"
  • Added some words in Security as per @afs

VladimirAlexiev and others added 17 commits August 5, 2025 10:22
@VladimirAlexiev
minor editorial change
5498be5
@VladimirAlexiev
plantuml version of diagram
8243c01
@VladimirAlexiev
add manual <color> and <u> tags to class names
3de3e44
@VladimirAlexiev
add manual <color> and <u> tags to class names
01b4b1b
@VladimirAlexiev @YoucTagh
Update shacl12-core/images/SHACL-UML.puml
138b174 
Co-authored-by: Yousouf Taghzouti <48333491+YoucTagh@users.noreply.github.com>
@VladimirAlexiev @YoucTagh
Update shacl12-core/images/SHACL-UML.puml
b991942 
Co-authored-by: Yousouf Taghzouti <48333491+YoucTagh@users.noreply.github.com>
@VladimirAlexiev
- sh:order: xsd:integer or xsd:decimal
72e0ff6 
- sh:path : predicate or list or blank node and link to #property-paths
- add ellipsis as per TallTed
- rdf:List -> "list" because SHACL doesn't require the rdf:List type
- add sh:nodeKind, sh:ignoredProperties
@VladimirAlexiev
add <color:blue><u> to PropertyGroup
1ef8993
@VladimirAlexiev
replace textual diagram with PlantUML SVG diagram
01fe8a2
@VladimirAlexiev
replace <img> with script that embeds SVG content
367e5a1
@VladimirAlexiev
Merge branch 'w3c:gh-pages' into gh-pages
e790e73
@VladimirAlexiev
Merge branch 'w3c:gh-pages' into gh-pages
8ade893
@VladimirAlexiev
Merge branch 'w3c:gh-pages' into gh-pages
620c4c9
@VladimirAlexiev
add core-components-list (3 fields), sh:targetWhere, sh:codeIdentifie…
2c4f8b4 
…r. Move sh:path to top of PropertyShape
@VladimirAlexiev
restore link to PropertyGroup
2770f32
@VladimirAlexiev
Merge branch 'w3c:gh-pages' into gh-pages
c9ab45c
@VladimirAlexiev
fixes w3c#163
8d3d823
@afs

afs commented Oct 23, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

a github mistake and this includes a gazillion commits.

PRs are merged, not single commits from a PR.

You can get the commit with, for example, git show 8d3d823 > F, then create a clean, up-to-date branch, git apply F and submit a PR with the required changes.

HolgerKnublauch
HolgerKnublauch requested changes Oct 23, 2025
View reviewed changes

@HolgerKnublauch HolgerKnublauch left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yeah I am afraid the best way forward would be for you to either create a new PR with just the change that you want, or roll back the unneeded parts here.

@afs

afs commented Jan 29, 2026

Copy link
Copy Markdown
Contributor

Ping @VladimirAlexiev

VladimirAlexiev and others added 8 commits February 6, 2026 11:54
@VladimirAlexiev @robert-david
plantuml version of diagram
b2f95c1
@VladimirAlexiev @robert-david
add manual <color> and <u> tags to class names
bf6370e
@VladimirAlexiev @robert-david
add manual <color> and <u> tags to class names
7b8e661
@VladimirAlexiev @YoucTagh @robert-david
Update shacl12-core/images/SHACL-UML.puml
2149bd5 
Co-authored-by: Yousouf Taghzouti <48333491+YoucTagh@users.noreply.github.com>
@VladimirAlexiev @YoucTagh @robert-david
Update shacl12-core/images/SHACL-UML.puml
68f2723 
Co-authored-by: Yousouf Taghzouti <48333491+YoucTagh@users.noreply.github.com>
@VladimirAlexiev @robert-david
- sh:order: xsd:integer or xsd:decimal
e62ae65 
- sh:path : predicate or list or blank node and link to #property-paths
- add ellipsis as per TallTed
- rdf:List -> "list" because SHACL doesn't require the rdf:List type
- add sh:nodeKind, sh:ignoredProperties
@VladimirAlexiev @robert-david
add <color:blue><u> to PropertyGroup
16a8d75
@VladimirAlexiev @robert-david
replace textual diagram with PlantUML SVG diagram
46821b5
caribouW3 and others added 20 commits February 9, 2026 17:06
@caribouW3 @robert-david
Create auto-publish-shacl12-rules.yml
bfdebba 
Auto-publication configuration
@caribouW3 @robert-david
Revert "Create auto-publish-shacl12-rules.yml"
2532618
@afs @robert-david
Auto-publish for rules and node-expr
01d8d64
@HolgerKnublauch @afs
@TallTed @robert-david
* w3c#676: Rewrote security section and used MUST for now
8ace8de 
* Update shacl12-sparql/index.html

Co-authored-by: Andy Seaborne <andy@apache.org>

* Switched to informative "should", accepted most of Ted's prose changes.

* Update shacl12-sparql/index.html

Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>

* Update shacl12-sparql/index.html

Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>

---------

Co-authored-by: Andy Seaborne <andy@apache.org>
Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>
@afs @robert-david
w3cGH-728: Inference graph
35f57f0
@simonstey @robert-david
Apply suggestion from @simonstey
8c2fec0
@HolgerKnublauch @robert-david
w3c#276: Fixed prefix declarations in test cases and adjusted example…
4ea7a61 
…s to use .com instead of .org (w3c#605)
@HolgerKnublauch @afs
@TallTed @robert-david
Issue 737 (w3c#750)
d1bebc1 
* w3c#737

* Added change log entry

* Update shacl12-core/index.html

Co-authored-by: Andy Seaborne <andy@apache.org>

* Update shacl12-core/index.html

Co-authored-by: Andy Seaborne <andy@apache.org>

* Update shacl12-core/index.html

Co-authored-by: Andy Seaborne <andy@apache.org>

* Apply suggestion from @afs

Co-authored-by: Andy Seaborne <andy@apache.org>

* Update shacl12-core/index.html

Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>

* Update shacl12-core/index.html

Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>

* Update shacl12-core/index.html

Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>

---------

Co-authored-by: Andy Seaborne <andy@apache.org>
Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>
@HolgerKnublauch @TallTed @robert-david
w3c#657: Clarified that members of list must be literals/IRIs and the…
a59ae1b 
… same order is kept (w3c#710)

* w3c#657: Clarified that members of list must be literals/IRIs and the same order is kept

* Update shacl12-node-expr/index.html

Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>

* Update shacl12-node-expr/index.html

Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>

* Switched from ex:concat to sparql:concat

* Apply suggestions from code review

Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>

* Update shacl12-node-expr/index.html

Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>

* Update shacl12-node-expr/index.html

Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>

* Update shacl12-node-expr/index.html

Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>

---------

Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>
@afs @robert-david
w3cGH-743: Remove some SRL functions
ff549e9
@afs @robert-david
More SHACL Rules syntax tests
e56c512
@afs @robert-david
Wellformedness, stratification, eval tests
e24c215
@afs @robert-david
RDF domain and range eval tests
ae93761
@YoucTagh @robert-david
Revise SHACL 1.2 Profiling abstract
bf87aa7
@afs @robert-david
Define negation element evaluation
3eac54a
@VladimirAlexiev @robert-david
replace <img> with script that embeds SVG content
7a2a755
@VladimirAlexiev @robert-david
plantuml version of diagram
d2cd519
@robert-david
Reset to gh-pages version
aa0b365
@robert-david
Removed redundant text
8ddce65
@robert-david
Merge branch 'gh-pages' into issue-163-PATH
7a393be
@robert-david

robert-david commented Feb 9, 2026

Copy link
Copy Markdown
Contributor

The rebase went wrong. I hope i cleaned it up correctly now.

@afs

afs commented Feb 9, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

The PR changes shacl12-core, shacl12-profiling and shacl12-sparql.
For example, the shacl12-core changes the shapes diagram and text related to it.

But the issue is only for shacl12-sparql.

It might be simpler to extract the changes related to $PATH (shacl12-sparql/index.html only) and make a new PR.

@nicholascar nicholascar added Core For SHACL 1.2 Core spec SPARQL For SHACL 1.2 SPARQL extensions spec Profiles For SHACL 1.2 Profiles spec labels May 28, 2026
@afs afs removed their request for review May 28, 2026 10:21
@nicholascar nicholascar removed Core For SHACL 1.2 Core spec Profiles For SHACL 1.2 Profiles spec labels May 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@afs afs afs left review comments

@bergos bergos Awaiting requested review from bergos

@YoucTagh YoucTagh Awaiting requested review from YoucTagh

@HolgerKnublauch HolgerKnublauch Awaiting requested review from HolgerKnublauch

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

SPARQL For SHACL 1.2 SPARQL extensions spec

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Clarify the use of $PATH, or string interpolation of SHACL paths