-
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
add episodes on estimate severity #1
Conversation
@pratikunterwegs thank you for your feedback. I added your specific edit suggestions to most of them. I have some open replies I would like to prioritize and request your re-review. Here are the links to each of them:
Open to discussing any misunderstanding. |
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.
Thanks @avallecam, this looks good to me. I've resolved the conversations as well, so feel free to merge once Carmen gives her fedback.
Thanks! I'll collect Carmen's feedback in a separate PR |
This PR adds episode content for the middle task 3 (MT3)
I'll appreciate your specific feedback providing a clear next step to remove, change or add content.
some questions to guide feedback are:
{cfr}
?how to review?
follow the steps in the reviewer role section of the contributing.md file
Before running
sandpaper::build_lesson()
, in theconfig.yaml
file deactivate these episodes:if local rendering doesn't work, please, report them in Issues.
you can read online the knitted content of this PR in branch
md-output-1
(drawback, figure captions not visible)