-
Notifications
You must be signed in to change notification settings - Fork 117
[HLS-360][WIP] Move docs over to readthedocs/glow #16
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
Conversation
Signed-off-by: kianfar77 <[email protected]>
Signed-off-by: kianfar77 <[email protected]>
* oops * fix pipe transformer cleanup Signed-off-by: kianfar77 <[email protected]>
Signed-off-by: kianfar77 <[email protected]>
Signed-off-by: kianfar77 <[email protected]>
Signed-off-by: kianfar77 <[email protected]>
Signed-off-by: kianfar77 <[email protected]>
Signed-off-by: kianfar77 <[email protected]>
Signed-off-by: kianfar77 <[email protected]>
Signed-off-by: kianfar77 <[email protected]>
Signed-off-by: kianfar77 <[email protected]>
Signed-off-by: kianfar77 <[email protected]>
Signed-off-by: Henry D <[email protected]> Signed-off-by: kianfar77 <[email protected]>
* Add glue fns Signed-off-by: Karen Feng <[email protected]> * Address comments Signed-off-by: Karen Feng <[email protected]> Signed-off-by: kianfar77 <[email protected]>
Signed-off-by: kianfar77 <[email protected]>
henrydavidge
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks reasonable to me as a first cut. Feel free to merge after addressing comments, and we can patch it up.
Also, could you remove the .. include:: /shared/products.rst from the bottom of each page?
henrydavidge
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adding explicit approval
Signed-off-by: kianfar77 <[email protected]>
…rojectglow#16) * Flatten info fields and include sample ids by default in VCF reader * add log4j.properties * add test for default schema Signed-off-by: Henry Davidge <[email protected]>
* Fix Glow tests (projectglow#6) * oops * fix pipe transformer cleanup Signed-off-by: kianfar77 <[email protected]> * Update log4j.properties Signed-off-by: kianfar77 <[email protected]> * livehtml Signed-off-by: kianfar77 <[email protected]> * fix Signed-off-by: kianfar77 <[email protected]> * more docs Signed-off-by: kianfar77 <[email protected]> * more docs Signed-off-by: kianfar77 <[email protected]> * Add license header (projectglow#9) Signed-off-by: kianfar77 <[email protected]> * add CONTIBURING.md (projectglow#10) Signed-off-by: Henry D <[email protected]> Signed-off-by: kianfar77 <[email protected]> * notebook addresses Signed-off-by: kianfar77 <[email protected]> * glow to Glow Signed-off-by: kianfar77 <[email protected]> * more docs Signed-off-by: kianfar77 <[email protected]> * sidebar width Signed-off-by: kianfar77 <[email protected]> * utility Signed-off-by: kianfar77 <[email protected]> * [HLS-353] Add utility function docs (projectglow#12) * Add glue fns Signed-off-by: Karen Feng <[email protected]> * Address comments Signed-off-by: Karen Feng <[email protected]> Signed-off-by: kianfar77 <[email protected]> * comments addressed Signed-off-by: kianfar77 <[email protected]> Signed-off-by: Henry Davidge <[email protected]>
This PR cleans and moves the docs to glow.
WIP.
Most of the text is polished.
Linked notebooks needs to be polished.