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

dcp-954 fix hca_bionetwork schema url #1526

Open
wants to merge 2 commits into
base: staging
Choose a base branch
from

Conversation

amnonkhen
Copy link
Collaborator

@amnonkhen amnonkhen commented Jun 29, 2023

schema url (describedBy) pattern was incorrect.

ebi-ait/dcp-ingest-central#954
Fixes: #1527
Related to: #1517

ESapenaVentura and others added 2 commits May 22, 2023 11:23
Release from staging to master 2023-05-22
schema url (`describedBy`) pattern was incorrect
@amnonkhen amnonkhen self-assigned this Jun 29, 2023
@amnonkhen amnonkhen requested review from NoopDog and hannes-ucsc June 30, 2023 08:51
@amnonkhen amnonkhen changed the base branch from master to staging June 30, 2023 08:54
@amnonkhen amnonkhen requested a review from ncalvanese1 June 30, 2023 13:41
Copy link

@ncalvanese1 ncalvanese1 left a comment

Choose a reason for hiding this comment

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

LGTM!

@amnonkhen amnonkhen changed the title dcp-954 fix hca_bionetwork schema url dcp-954 fix hca_bionetwork schema url (expedited) Jun 30, 2023
Copy link
Collaborator

@ESapenaVentura ESapenaVentura left a comment

Choose a reason for hiding this comment

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

It's missing writing the changes in the update_log.csv - Also not sure if the human-readable markdowns have been generated

Here is a document with the specifics on what is expected that is done during an update https://github.com/HumanCellAtlas/metadata-schema/blob/master/docs/committers.md#specific-how-to-for-making-changes, but we can discuss on monday if it's unclear!

Copy link
Contributor

@hannes-ucsc hannes-ucsc left a comment

Choose a reason for hiding this comment

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

Title is misleading. This cannot be an expedited review. See section 1.6.1 of the SOP.

Copy link
Contributor

@NoopDog NoopDog left a comment

Choose a reason for hiding this comment

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

Nice ! ⭐

@amnonkhen amnonkhen changed the title dcp-954 fix hca_bionetwork schema url (expedited) dcp-954 fix hca_bionetwork schema url Jul 20, 2023
@amnonkhen amnonkhen requested a review from hannes-ucsc July 20, 2023 14:07
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.

5 participants