-
Notifications
You must be signed in to change notification settings - Fork 68
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
[SYNPY-1334] Revamp getting started docs #1036
Conversation
Hello @BryanFauble! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2024-01-02 20:25:51 UTC |
docs/tutorials/sample_files/my_ad_project/single_cell_RNAseq_batch_1/SRR12345678_R2.fastq.gz
Outdated
Show resolved
Hide resolved
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.
🔥 LGTM! I'm going to pre-approve because we can always continuously improve documentation.
It will be continued iteration.
Co-authored-by: Thomas Yu <[email protected]>
Co-authored-by: Thomas Yu <[email protected]>
…ub.com/Sage-Bionetworks/synapsePythonClient into SYNPY-1334-revamp-getting-started-docs
Quality Gate passedThe SonarCloud Quality Gate passed, but some issues were introduced. 152 New issues |
Problem:
Solution:
Project
,Folder
,File
tutorials. I'd like feedback around the flow of these tutorials and what I might be able to improve.Further Reading
section of the doc pages that we can expand on the Models/Concepts of the system. This was specifically done so I can link to them from the Tutorial section and I didn't need to crowd the tutorials with additional context around these concepts.