-
Notifications
You must be signed in to change notification settings - Fork 565
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
Infixowl coverage ii #2024
Infixowl coverage ii #2024
Conversation
…rror" suffix to MaformedClass and stringify OWL namespace where necessary for `split`.
…or `valid_restr_props`, adjust various methods to handle None, add informative docucomments w.r.t. Attempto integration.
@gjhiggins looks good, but I'm going to add release notes also. |
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.
Changes look good to me, please have a look at the changes I included for readme and the exception to see if you are fine with it.
Yep, fine by me. Extended documentation to come (honest, guv) |
Summary of changes
Final stage of update to infixowl, removing unused code, tidying, adding some docucomments, minor improvements in robustness, updating previously-failing tests, adding more tests. Expanded documentation to follow.
Checklist
./examples
for new features.CHANGELOG.md
).so maintainers can fix minor issues and keep your PR up to date.