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
Implement Feature/rdf #4
Conversation
Also added 5f47bfd and later, might be good to check those in isolation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Got this running so I assume it's fine. Didn't manage to do the whole curl streaming into the application due to the whole having to set NODE_PATH variable also.
I also don't think it's really feasible to do a full code review of everything. There's so much boilerplate stuff (e.g. the jsonld files, the similar interfaces, etc.) making it hard to see the actual relevant parts to look at :|
export class ActorInitRdfParse extends ActorInit implements IActorInitRdfParseArgs { | ||
|
||
public readonly mediatorRdfParse: Mediator<Actor<IActionRdfParseOrMediaType, IActorTest, | ||
IActorOutputRdfParseOrMediaType>, IActionRdfParseOrMediaType, IActorTest, IActorOutputRdfParseOrMediaType>; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Totally readable :P
"no-var-requires": false, | ||
"no-unused-expression": false | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:o
(and some others)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Most files have this apparently, will fix once this is merged.
Related to rdfjs/N3.js#120
196325a
to
8c9a671
Compare
Packages related to RDF parsing