Skip to content
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

Ntr allelopath issue809 #815

Merged
merged 7 commits into from
Jan 27, 2025
Merged

Conversation

mdrishti
Copy link
Contributor

Added ID ranges for mdrishti and inverse term 'has alleopath' for 'alleopath of'

@mdrishti mdrishti requested a review from anitacaron as a code owner October 15, 2024 14:14
@anitacaron anitacaron linked an issue Oct 15, 2024 that may be closed by this pull request
@mdrishti
Copy link
Contributor Author

Hi @anitacaron @bpeters42 . I added a set of ID ranges and the inverse term 'has allelopath' for 'allelopath of'. Let me know if my pull request fulfills the requirements - awaiting review.

Regards,
Disha

src/ontology/ro-edit.owl Outdated Show resolved Hide resolved
@mdrishti
Copy link
Contributor Author

Hi @anitacaron, I have made the changes requested by @cmungall . Let me know how to proceed further with this pull-request.

Copy link
Member

@balhoff balhoff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@balhoff balhoff dismissed cmungall’s stale review January 27, 2025 16:17

I think the request was addressed.

@balhoff balhoff force-pushed the ntr_allelopath_issue809 branch from 4d17cd6 to b93d3c3 Compare January 27, 2025 16:29
Copy link
Collaborator

@anitacaron anitacaron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Technically, it's good to me.

@anitacaron anitacaron merged commit aebaf6a into oborel:master Jan 27, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

NTR: has allelopath
5 participants