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

Pedantic mode #57

Merged
merged 5 commits into from
Nov 8, 2019
Merged

Pedantic mode #57

merged 5 commits into from
Nov 8, 2019

Conversation

stain
Copy link
Contributor

@stain stain commented Nov 1, 2019

Working with @fbacall we found various issues with @id and identifier. Some of the examples were outdated.

@stain
Copy link
Contributor Author

stain commented Nov 8, 2019

editor call with @ptsefton 2019-11-08:

Agreed that ./foo should be foo

Still to be investigated foo/ vs foo for directories. Calcyte uses foo.

Blank nodes ARE permitted, modify "MUST URI" to "SHOULD URI". # ones are OK, but also permit _: ones as MAY. # example changes OK. Not too important that ro-crate-metadata.jsonld#tim != ro-crate-preview.html#tim as it's still better than blank nodes which are always different.

@stain
Copy link
Contributor Author

stain commented Nov 8, 2019

OK, so the remaining contentious part here is directories should end in / or not and if we should say something about it. I'll raised that as new issue.

@stain stain merged commit 2d23daf into master Nov 8, 2019
@stain stain deleted the pedantic-mode branch November 8, 2019 16:36
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.

None yet

1 participant