-
Notifications
You must be signed in to change notification settings - Fork 5
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
Added CLI tools documentation #84
Conversation
Codecov Report
@@ Coverage Diff @@
## main #84 +/- ##
=======================================
Coverage 99.07% 99.07%
=======================================
Files 59 59
Lines 2366 2381 +15
=======================================
+ Hits 2344 2359 +15
Misses 22 22
Continue to review full report at Codecov.
|
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.
Please make sure that the RML mapping file is called "mappings.rml.ttl" and not "output.rml.ttl" in the code (and make sure the CLI tools work with those names :) ).
See for example:
58b5831#diff-e92768c2ab48cd9b2c5d4006f7fb418eca719f56c8da494add0f11149532ee14
and
https://github.com/meaningfy-ws/ted-sws/pull/84/files#diff-c6928e509891b77e660da91efc11c53fce305cdd32c9d49349b75d2e28ec877f
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.
Approve it for now, but the RML file name issue pointed out before should be resolved in the next round.
No description provided.