search

monarch-initiative / omim

Data ingest pipeline for OMIM.
6 stars 2 forks source link

Remove `omim.ttl` from commits? #95

Closed joeflack4 closed 1 year ago

joeflack4 commented 1 year ago

Overview

I think we had previously commit it for convenience, but now especially with weekly releases, I'm not sure that's helpful.

matentzn commented 1 year ago

I think it's fine to commit it, this gives us a proper version history of the file! The important thing is to recreate it each time!

joeflack4 commented 1 year ago

I suppose that's true, but right now the GitHub action doesn't commit it; just makes a release.

matentzn commented 1 year ago

In this case: delete for sure

joeflack4 commented 1 year ago

Completed by: https://github.com/monarch-initiative/omim/commit/da7c795b2149824c84a8d1e15234b7ab7f6c7b42