-
Notifications
You must be signed in to change notification settings - Fork 8
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 the SemanticArtefact class definition to MOD 2.0 #55
Conversation
Hi @biswanathdutta, would you be able to review this PR please? Thanks |
There are issues. So dismissing the first review. I am going to provide my specific remarks.
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.
@agbeltran we should add SemanticArtefact description. Also, add its subClassOf information.
The definition may be moved to the class section (below line number 2981) for its better management.
thanks @biswanathdutta - I added the definition and moved it to the classes section - please note that this is a fix of V2.0 but this will soon be replaced with version 3.0 - it is only to address the issue raised for this version. I would like to merge this asap. |
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.
looks fine to me
thanks @antony-wilson |
Resolving issue #40