search

ga4gh-discovery / ga4gh-service-info

GA4GH service-info specification.
Apache License 2.0
9 stars 8 forks source link

Adding maturity field #43

Closed andrewyatz closed 5 years ago

andrewyatz commented 5 years ago

Issue #33 raised a need for a maturity field. This adds that field in, types it to an enum and makes it a required field. The enum requirement could be removed at a later date.

jrambla commented 5 years ago

The ones requiring it are the clients of the service, not the owners. It must be required if it should be of any utility, on the contrary we could end up in the same situation that we are today. Forcing to choose an option would make the owners to think about it.

andrewyatz commented 5 years ago

I agree with @jrambla that it should be required and why I added unknown in to give those implementing it a way to shortcut the response. If we were to remove unknown then I push for it to be optional

mcupak commented 5 years ago

Agreed to rename to environment and replace unknown with null (i.e. make optional).