search

OBOFoundry / purl.obolibrary.org

A system for managing OBO PURLs
BSD 3-Clause "New" or "Revised" License
77 stars 128 forks source link

Update omo.yml #847

Closed zhengj2007 closed 2 years ago

zhengj2007 commented 2 years ago

Added removed version IRI back in

zhengj2007 commented 2 years ago

@matentzn I think we should added the version IRI for latest OMO release file too.

jamesaoverton commented 2 years ago

Many projects prefer to use a general prefix for releases, and not add an explicit exact entry for each release. I'm not sure which is really better, but it's better to stick with a single approach for a given project.

matentzn commented 2 years ago

Thank you for catching that @jamesaoverton.

Yes, @zhengj2007 the Version IRIs follow the releases pattern, see here.

The version IRI resolves fine, and we do not have to add it to config manually ever again: http://purl.obolibrary.org/obo/omo/releases/2022-04-27/omo.owl

Note that the these past versions that @zhengj2007 added here followed a different pattern: https://github.com/information-artifact-ontology/ontology-metadata/commit/3951ba18423423498584e2dd24f67e5514dd6b6f

So there actually is a case to merge this pull request as a one-off to ensure these two past version IRIs resolve.

matentzn commented 2 years ago

@zhengj2007 as an aside (I also adhere to that in 99.9% of the cases) - no one should merge changes to the purl config other than James.. This avoids for example miscommunication leading to a premature merge which breaks the whole PURL system.