-
Notifications
You must be signed in to change notification settings - Fork 10
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
Updated ontoconvert help #715
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #715 +/- ##
==========================================
+ Coverage 71.79% 71.82% +0.02%
==========================================
Files 16 16
Lines 3429 3429
==========================================
+ Hits 2462 2463 +1
+ Misses 967 966 -1 ☔ View full report in Codecov by Sentry. |
@@ -86,13 +86,15 @@ def main(argv: list = None): | |||
action="store_true", | |||
help="Do not infer imported ontologies.", | |||
) | |||
# To be implemented... |
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.
# To be implemented... | |
# To be implemented...Write an issue about it and refer to that issue here |
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.
Added issue #716
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.
OK
Description
Seems that the
--base-iri
option ofontoconvert
is not implemented. Added a note about that in the--help
documentation.This option is used in some EMMO workflows, so we keep it to not break them.
This is a place where we really should have added a "TODO" comment in the code - but pylint doesn't accept that.
Type of change
Checklist
This checklist can be used as a help for the reviewer.
Comments