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

Mmgh #4

Merged
merged 2 commits into from Feb 25, 2015
Merged

Mmgh #4

merged 2 commits into from Feb 25, 2015

Conversation

no-reply
Copy link
Owner

Deals with the lexical issue raised by #2, but does not yet add EBNF.

Also, closes #3.

Tom Johnson added 2 commits February 25, 2015 10:38
Update with new mmgh? line ending.
0fae3aa
Add #way_of_the_force
a507d4b 
Warns about the dangers of dark-side serializations.
no-reply pushed a commit that referenced this pull request Feb 25, 2015
Merge pull request #4 from no-reply/mmgh
f7e26ae 
Mmgh
@no-reply no-reply merged commit f7e26ae into master Feb 25, 2015
@anarchivist
Copy link
Contributor

@no-reply i think my suggestion with #3 is that it should call warn whenever you tried to call #dump on a serialization that was not :yodatriples. but i think this is a good first step.

@no-reply no-reply deleted the mmgh branch February 25, 2015 20:12
@no-reply
Copy link
Owner Author

So, require 'rdf/yoda_triples should reopen RDF::Writer to add warnings?

@anarchivist
Copy link
Contributor

Yes - will comment/reopen #3.

@anarchivist anarchivist mentioned this pull request Feb 25, 2015
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@no-reply @anarchivist