Skip to content

fixes for grammar, punctuation, etc. throughout#794

Open
TallTed wants to merge 1 commit intogh-pagesfrom
fix/TallTed-20260222-patch-1
Open

fixes for grammar, punctuation, etc. throughout#794
TallTed wants to merge 1 commit intogh-pagesfrom
fix/TallTed-20260222-patch-1

Conversation

@TallTed
Copy link
Member

@TallTed TallTed commented Feb 23, 2026

There's no issue for this.

The preview gets a 404 when I try to view it. I have no idea why.

@smessie
Copy link
Member

smessie commented Feb 23, 2026

The preview gets a 404 when I try to view it. I have no idea why.

You included the branch gh-pages instead of the directory name shacl12-rules, so working link: https://raw.githack.com/w3c/data-shapes/fix/TallTed-20260222-patch-1/shacl12-rules/index.html

<h2>Shape Rules Abstract Syntax</h2>

<p>The Shape Rules Abstract Syntax</p>
<p>The Abstract Syntax of Shape Rules</p>
Copy link
Contributor

Choose a reason for hiding this comment

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

The name is "Shape Rules Abstract Syntax". See <h2>.

Copy link
Member Author

Choose a reason for hiding this comment

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

Any chance you'd agree to go halfway, for both <h2> and this <p>, i.e., "Shape Rule Abstract Syntax"? The plural form in this word order is very UK English, but I think this doc (if not all of this WG) is US English

Suggested change
<p>The Abstract Syntax of Shape Rules</p>
<p>Shape Rule Abstract Syntax</p>

Copy link
Contributor

Choose a reason for hiding this comment

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

This is an unfinished paragraph.

Copy link
Member Author

Choose a reason for hiding this comment

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

This is an unfinished paragraph.

I'm not seeing any reason why that matters.

@TallTed
Copy link
Member Author

TallTed commented Feb 23, 2026

You included the branch gh-pages instead of the directory name shacl12-rules

Ah, thank you. Fixed.

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.

3 participants