-
Notifications
You must be signed in to change notification settings - Fork 66
Add rdfSaver Files #14
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
Conversation
0b39686 to
bd20335
Compare
Signed-off-by: abhishekspeer <[email protected]>
|
Hi @swinslow, I've added the code to convert intermediate data structure to standard data structure with this commit. Please have a look and let me know in case you expect any changes |
|
@abhishekspeer Open a separate PR for that. Since we want to merge that code with rdf-reader branch and see the reader part running. This PR is just for rdfsaver. |
|
Thanks @rtgdk! I've referenced a separate PR for that. Please have a look and let me know if there is anything I should change. :) |
Signed-off-by: abhishekspeer <[email protected]>
Signed-off-by: abhishekspeer <[email protected]>
Signed-off-by: abhishekspeer <[email protected]>
Signed-off-by: abhishekspeer <[email protected]>
Signed-off-by: abhishekspeer <[email protected]>
bd5741c to
b709a70
Compare
Signed-off-by: abhishekspeer <[email protected]>
|
@abhishekspeer Did you see that the DCO sign-off check is failing? It appears that some of the commits are missing the DCO sign-off (see the "Details" button next to the DCO line in the checks box). Can you please amend those commits or if necessary squash and re-submit as one commit with the DCO sign-off corrected? |
|
@swinslow I have made the required changes you suggested and DCO passes now. Please let me know if there is anything else I need to change. Thanks! :) |
|
Thanks @abhishekspeer! |
Signed-off-by: abhishekspeer [email protected]
RDF writer support is added via this pull request.