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.

Sign up for GitHub

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

Jump to bottom

Add informative quoted triples example, Closes #12 #13

Merged
merged 3 commits into from
Apr 21, 2023
Merged

Add informative quoted triples example, Closes #12 #13

merged 3 commits into from
Apr 21, 2023

Conversation

rubensworks
Copy link
Member

@rubensworks rubensworks commented Apr 7, 2023 •
edited by pr-preview bot
Loading

@rubensworks rubensworks requested review from kasei and afs April 7, 2023 13:13
@afs afs added the needs discussion Proposed for discussion in an upcoming meeting label Apr 13, 2023
@afs
Copy link
Contributor

afs commented Apr 13, 2023

Many readers interested in RDF-star will have read the CG report.

I am in favour of adding material about quoted triples so that readers can see the flow from CG work to WG final output.

TallTed
TallTed reviewed Apr 13, 2023
View reviewed changes
spec/index.html Outdated Show resolved Hide resolved
@rubensworks @TallTed
Update spec/index.html
ed6423a 
Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>
hartig
hartig reviewed Apr 13, 2023
View reviewed changes
spec/index.html Outdated Show resolved Hide resolved
@rubensworks @hartig
Update spec/index.html
f16e9f3 
Co-authored-by: Olaf Hartig <olaf.hartig@liu.se>
afs
afs approved these changes Apr 16, 2023
View reviewed changes
Copy link
Contributor

@afs afs left a comment

Choose a reason for hiding this comment

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

After the telecon of 2012-04-13, I believe we can go ahead and add quoted triple examples to specs.

hartig
hartig approved these changes Apr 16, 2023
View reviewed changes
Copy link
Contributor

@hartig hartig left a comment

Choose a reason for hiding this comment

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

I agree with @afs

@rubensworks rubensworks merged commit 890be5a into main Apr 21, 2023
@rubensworks rubensworks deleted the quoted-triples branch April 21, 2023 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TallTed TallTed TallTed left review comments

@gkellogg gkellogg gkellogg approved these changes

@afs afs afs approved these changes

@hartig hartig hartig approved these changes

@kasei kasei Awaiting requested review from kasei

Assignees
No one assigned
Labels
needs discussion Proposed for discussion in an upcoming meeting
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@rubensworks @afs @gkellogg @hartig @TallTed