-
Notifications
You must be signed in to change notification settings - Fork 2
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
Training template updates #175
Conversation
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! Only small (and not super necessary) comment is to maybe bold the key words/links in each bullet under Pre-workshop prep
? Just for consistency.
workshop/SCHEDULE.md
Outdated
<!--See an example from a past remote workshop here: https://github.com/AlexsLemonade/2023-may-training/blob/main/workshop/SCHEDULE.md --> | ||
<!--See an example from a past in-person workshop here: https://github.com/AlexsLemonade/2023-june-training/blob/main/workshop/SCHEDULE.md --> | ||
<!--See an example from a past remote workshop here: https://github.com/AlexsLemonade/2024-june-training/blob/main/workshop/SCHEDULE.md --> | ||
<!--See an example from a past in-person workshop here: https://github.com/AlexsLemonade/2024-december-training/blob/main/workshop/SCHEDULE.md --> | ||
|
||
|
||
| Time | Topic | |
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.
Maybe drop the trailing |
from the table here, just to avoid future formatting annoyances?
This PR closes a handful of small issues:
workshop/SCHEDULE.md
example links were updated with more recent workshop schedules that use the current version of this repoif
statement to not print the "new to R?" bullet on the home page if we're teaching an advanced workshop.