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.

Sign up for GitHub

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

Jump to bottom

Update 01-intro.Rmd with code chunk names and fix urls #95

Merged
merged 12 commits into from
Jan 13, 2025
Merged

Update 01-intro.Rmd with code chunk names and fix urls #95

merged 12 commits into from
Jan 13, 2025

Conversation

kweav
Copy link
Contributor

@kweav kweav commented Dec 11, 2024

Added code chunk names for the audience, topics covered, and learning objectives slides

@kweav
Update 01-intro.Rmd with code chunk names
06f82ef 
Added code chunk names for the audience, topics covered, and learning objectives slides
@kweav kweav requested a review from carriewright11 December 11, 2024 17:06
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Dec 11, 2024 •
edited
Loading

No spelling errors! 🎉
Comment updated at 2025-01-13-17:21:51 with changes from 4790863

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Dec 11, 2024 •
edited
Loading

⚠️ Check: broken url errors did not fully run! Go to the Actions tab to find more info.
Post issue to https://github.com/jhudsl/OTTR_Template/issues if this seems incorrect.
Comment updated at 2025-01-13-17:21:52 with changes from 4790863

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Dec 11, 2024 •
edited
Loading

Re-rendered previews from the latest commit:

* note not all html features will be properly displayed in the "quick preview" but it will give you a rough idea.

Updated at 2025-01-13 with changes from the latest commit 4790863

@kweav
Update exclude_files.txt
3942555 
included `getting_started.md` since it was flagging a lot of spelling errors
@kweav
Copy link
Contributor Author

kweav commented Dec 12, 2024

@carriewright11 going through the broken URLs, but I'll need your approval on the ones I find/switch out. I'll ping you when I've gone as far as I can.

kweav added 6 commits December 20, 2024 11:08
@kweav
Update 03-Binary_data_to_computations.Rmd
519bba7 
Update a link
@kweav
Update 03-Binary_data_to_computations.Rmd
393515b 
Point to history of the world for medium article
@kweav
Update book.bib
bdca9d0 
Update link for medium article here too
@kweav
Update 05-Shared_computing_etiquette.Rmd
7c148d6 
Update with strong passwords and SGE slide deck
@kweav
Update 06-General_Platforms.Rmd
6dfb521 
Update a few links here
@kweav
Create ignore-urls.txt
82165bd 
Try adding wiki link to an ignore urls file
carriewright11
carriewright11 approved these changes Jan 13, 2025
View reviewed changes
Copy link
Member

@carriewright11 carriewright11 left a comment

Choose a reason for hiding this comment

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

these look great - thank you!

@carriewright11 carriewright11 changed the title Update 01-intro.Rmd with code chunk names Update 01-intro.Rmd with code chunk names and fix urls Jan 13, 2025
@carriewright11
Copy link
Member

hmm not sure why url action is failing. I think I will merge this and troubleshoot that in another PR

@carriewright11 carriewright11 merged commit 6e5aff7 into main Jan 13, 2025
10 of 11 checks passed
@carriewright11 carriewright11 deleted the kweav-nameChunks branch January 13, 2025 17:36
@carriewright11 carriewright11 mentioned this pull request Jan 13, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@carriewright11 carriewright11 carriewright11 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kweav @carriewright11