OBOFoundry / OBOFoundry.github.io

Metadata and website for the Open Bio Ontologies Foundry Ontology Registry
http://obofoundry.org
Other
166 stars 205 forks source link

Document update NOR dashboard automatic workflow better #2243

Closed matentzn closed 1 year ago

matentzn commented 1 year ago

There is a GitHub action that automatically rebuilds all the metadata required for the functioning of the site.

Example https://github.com/OBOFoundry/obo-nor.github.io/pull/21/files

- When dashboard config is updated, and every Monday morning, the OBO NOR Dashboard is regenerated using [this](provide a link) GitHub action. The OBO NOR Manager should sanity check the diff for any oddities, in particular changes to the file `dashboard/dashboard-results.yml`.  This PR should always be reviewed by the NOR Manager and not be automatically merged into the main branch as part of the GitHub action.
pfabry commented 1 year ago

Just to be sure, I understand I must change the following workflow and replace your github ID on the last line. However, I think I don't have write access on the NOR repo. Could that be a problem for the update action ?

matentzn commented 1 year ago

You should change this line:

https://github.com/OBOFoundry/obo-nor.github.io/blob/master/.github/workflows/dashboard.yml#L37

matentzn commented 1 year ago

I gave everyone access to that repo now - was an oversight.

pfabry commented 1 year ago

cf. #2248

In addition, do we still want to ask new ontology requestors to join the obo-discuss mailing list vs keeping all communications in the issue or using Slack ?

matentzn commented 1 year ago

Great catch @pfabry, see https://obofoundry.org/faq/how-do-i-register-my-ontology.html, they absolutely have to announce their submission on obo-discuss as well. Can you add this to your duties:

Slack: I think you can add this to your list:

pfabry commented 1 year ago

cf. new commit on #2248

pfabry commented 1 year ago

2248 has been merged.