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

Update workshop-resources file #129

Merged
merged 10 commits into from
Sep 13, 2023

Conversation

sjspielman
Copy link
Member

@sjspielman sjspielman commented Aug 23, 2023

⚠️ Stacked on #128
Closes #122

This PR updates the workshop/workshop-resources.md file.

I deleted content that was moved out into the new bulk & single-cell resources files created in additional-resources as part of #127, and created sections here to link those files (as well as the R resources file) from here. I also cleaned up some text in this markdown to make it more focused on resources and less on consultations, added all the cheatsheets, and fixed a relative link that became deprecated.

@sjspielman sjspielman changed the title Overhaul resources file; delete old content, link to new additional-r… Update workshop-resources file Aug 23, 2023
- Call directory website, not main
- Bump PR action to @5, remove safe directory creation
- Stop curling, and start copying, because _of course_
- Add bulk RNAseq
- Fancier PR comment
- Use case statement to simplify the code while still only getting a given workshop's files
…cp line. Tested locally as script and syntax is good.
Base automatically changed from sjspielman/121-completed-notebooks-virtual-setup to template-refresh August 25, 2023 19:40
@sjspielman
Copy link
Member Author

sjspielman commented Sep 7, 2023

Requesting review now, but not a priority to get to it this sprint!

@sjspielman sjspielman requested a review from jashapiro September 7, 2023 13:22
sjspielman and others added 2 commits September 7, 2023 09:30
Co-authored-by: Joshua Shapiro <josh.shapiro@ccdatalab.org>
…ted-notebooks-gha

Add GHA to copy completed notebooks
@sjspielman
Copy link
Member Author

Noting I merged in the GHA to copy completed notebooks, but should not cause "clutter" for review here since only one file! So, no need to review .github/workflows/copy-completed-notebooks.yml here (unless you see something that we missed!)

Copy link
Member

@jashapiro jashapiro left a comment

Choose a reason for hiding this comment

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

LGTM

@sjspielman sjspielman merged commit a8863f8 into template-refresh Sep 13, 2023
@sjspielman sjspielman deleted the sjspielman/122-workshop-resources branch September 13, 2023 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants