-
Notifications
You must be signed in to change notification settings - Fork 51
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
Applying changes for: Remove metazoan taxon constraint for stem cell #2699
base: master
Are you sure you want to change the base?
Conversation
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.
No objection, but I note that if we remove that restriction, then we should also remember to remove the existing comment which states that
This term applies to metazoan. For plant stem cells, consider using PO:0004011 ‘initial cell’ or its parent PO:0004010 ‘meristematic cell’.
and also change the “animal stem cell” synonym from exact
to narrow
.
This PR has not seen any activity in the past month; if nobody comments or reviews it in the next week, the PR editor will be allowed to proceed with merging without explicit approval, should they wish to do so. |
Are the CL developers okay with merging this PR? |
Since "stem cell" is no longer taxon-restricted to metazoan, the comment advising against its use for non-metazoan is no longer relevant. Also "animal stem cell" is no longer an exact synonym.
I updated the PR to remove the now obsolete comment I mentioned earlier. I am personally fine with the change. Don’t know the opinion of other CL folks, but if nobody weighs in, I’d suggest raising the topic at the next CL call. |
The following commands were executed:
* delete edge CL:0000034 RO:0002162 NCBITaxon:33208
Fixes #2693