Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add-new-source.md updates #307

Merged
merged 1 commit into from
May 18, 2023
Merged

add-new-source.md updates #307

merged 1 commit into from
May 18, 2023

Conversation

joeflack4
Copy link
Contributor

  • Docs: Updated docs/developer/add-new-source.md: Created sub-sections, added some additional information, and updated step 1 info.

Copy link
Contributor Author

@joeflack4 joeflack4 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hrsdhgd @souzadevinicius FYI / if you have time to review

docs/developer/add-new-source.md Show resolved Hide resolved

### 1.2. `update_repo`
From `src/ontology`, run `sh run.sh make update_repo`.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@matentzn Added this next step about update_repo

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, good!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added a note about it needing to run twice as well. However, as I mentioned in this comment, running it a second time did not seem to do anything for me.

## 2. `metadata/*.yaml`
Add a new metadata file to [src/ontology/metadata](https://github.com/monarch-initiative/mondo-ingest/blob/main/src/ontology/metadata). It is important we try and document as much about the source as we can.

## 3. `mkdocs.yaml`
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@matentzn Do you think something is missing here? I noticed that it looks like I manually needed to add links to gard.md in several places. It doesn't look like mkdocs build is what I want to run. Perhaps we need to write something here about manually making some other additions? Such as:

  1. sources/ONTOLOGY.md
  2. Adding ONTOLOGY to sources.md
  3. What about metrics/ONTOLOGY.md as well as adding a link to that file in mkdocs.yaml as well?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unfortunately ues, mkdocs.yaml is entirely automatically managed. So all of these need to be added.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Np. I added instructions for adding these. I see now that there are some goals that help with these:
../../docs/sources/%.md
../../docs/metrics/%.md

@joeflack4 joeflack4 added the documentation Improvements or additions to documentation label May 16, 2023

### 1.2. `update_repo`
From `src/ontology`, run `sh run.sh make update_repo`.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, good!

docs/developer/add-new-source.md Show resolved Hide resolved
## 2. `metadata/*.yaml`
Add a new metadata file to [src/ontology/metadata](https://github.com/monarch-initiative/mondo-ingest/blob/main/src/ontology/metadata). It is important we try and document as much about the source as we can.

## 3. `mkdocs.yaml`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unfortunately ues, mkdocs.yaml is entirely automatically managed. So all of these need to be added.

@souzadevinicius
Copy link
Member

@hrsdhgd @souzadevinicius FYI / if you have time to review

LGTM

@joeflack4 joeflack4 mentioned this pull request May 16, 2023
@joeflack4 joeflack4 force-pushed the docs-new-sources branch 4 times, most recently from 67ec741 to 693196c Compare May 17, 2023 23:55
…s, added some additional information, and updated step 1 info.
@joeflack4 joeflack4 merged commit 7853edb into main May 18, 2023
@joeflack4 joeflack4 deleted the docs-new-sources branch May 18, 2023 00:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants