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 _bookdown.yml #115

Merged
merged 7 commits into from
Jun 14, 2024
Merged

Update _bookdown.yml #115

merged 7 commits into from
Jun 14, 2024

Conversation

kweav
Copy link
Contributor

@kweav kweav commented Jun 7, 2024

Update the book_filename and repo within _bookdown.yml

Update the `book_filename` and `repo` within `_bookdown.yml`
@kweav kweav requested a review from ehumph June 7, 2024 18:23
Copy link
Contributor

github-actions bot commented Jun 7, 2024

⚠️ broken url errors ⚠️
There are broken url errors that need to be addressed.
Click here ➡️ for broken url errors!
Add errors that aren't errors to the resources/ignore-urls.txt file of this repo.
If you are having troubles see this guide
Comment updated at 2024-06-07-22:43:49 with changes from 96703d8

Copy link
Contributor

github-actions bot commented Jun 7, 2024

No spelling errors! 🎉
Comment updated at 2024-06-07-22:33:46 with changes from 96703d8

kweav added 5 commits June 7, 2024 15:03
Add words flagged by most recent spell check
Fix a broken link that pointed to old course title
Removing links to a PDF that is no longer available in the docs > coursera > example chapter markdown
Update `index.Rmd` to point to the new GitHub repo and bookdown website following the name change. Also added the coursera link
Point to issues in new repo following name change
@kweav
Copy link
Contributor Author

kweav commented Jun 7, 2024

Rerunning the workflow for outputting the courses because several of the broken links are using iframes pointing to the old course name.

Copy link
Contributor

github-actions bot commented Jun 7, 2024

Re-rendered previews from the latest commit:

Updated at 2024-06-07 with changes from 96703d8

@kweav
Copy link
Contributor Author

kweav commented Jun 10, 2024

@ehumph I have addressed flags from the spelling and URL checkers and have re-rendered the output courses and think this is ready for review now. Not sure if the course outputs will need to be re-rendered after merging the changes to address the rest of the broken URLs -- but I think @carriewright11 can help with determining that

@carriewright11 carriewright11 merged commit d8c8ea9 into main Jun 14, 2024
12 of 13 checks passed
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