monarch-initiative / mondo-ingest

Coordinating the mondo-ingest with external sources
https://monarch-initiative.github.io/mondo-ingest/
6 stars 3 forks source link

Remove duplicate prefix maps in `prefixes.csv` #503

Closed joeflack4 closed 5 months ago

joeflack4 commented 5 months ago

Overview

Removed some duplicate entries (same prefix and URI prefix) in prefixes.csv.

Pre-merge checklist

Documentation

Was the documentation added/updated under docs/?

QC

Was the full pipeline run before submitting this PR using sh run.sh make build-mondo-ingest on this branch (after docker pull obolibrary/odkfull:dev), and no errors occurred?

New Packages

Were any new Python packages added?

Were any other non-Python packages added?

PR Review and Conversations Resolved

Has the PR been sufficiently reviewed by at least 1 team member of the Mondo Technical team and all threads resolved?

Changes

joeflack4 commented 5 months ago

@twhetzel Despite Nico's comment above, he approved this because at first he thought I was removing non-duplicate cases.

I marked 'no' on QC because there should be no effective changes here that should make any difference to any code, so I'm going to merge now!