-
Notifications
You must be signed in to change notification settings - Fork 47
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
2024-02-13 Release #788
2024-02-13 Release #788
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.
@anitacaron I think there is a problem with the changes from #761. @pgaudet did not mean to put the terms under 'regulates' (please let me know if I'm misreading the diff). Is there time to fix that before release?
@balhoff, that's fine. I'll wait to fix this in another PR. But I've checked the issue linked to that PR, and it's consistent. |
@anitacaron Most everything looks fine. However, I noticed that I am not included in the orcidio import, although I added myself as part of the owns/owned by PR. |
@wdduncan sorry, someone needs to approve that PR first 😬 |
Thanks @anitacaron. I pinged some folks to review. |
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 great!
@wdduncan, no worries, all changes to add you into orcidio_import are in your PR. |
No description provided.