Closed jaclyn-taroni closed 2 years ago
AppVeyor build 1.0.441 for commit 047858aa2813b1eef043f5a6fd339e923d4ce1ad is now complete. Found 5 potential spelling error(s). Preview:
content/02.intro.md:5:TODO
content/02.intro.md:8:TODO
content/02.intro.md:13:subtypes
content/02.intro.md:17:hierchical
content/02.intro.md:18:subtype...
AppVeyor build 1.0.442 for commit 5d0ae6d3f007a19a0deaa9b639b50516f25497fe is now complete. Found 2 potential spelling error(s). Preview:
content/02.intro.md:5:TODO
content/02.intro.md:8:TODO...
AppVeyor build 1.0.445 for commit 4e652f9bb16ee64aa07dc86e1593846ad965c2ac is now complete. Found 1 potential spelling error(s). Preview:
content/02.intro.md:8:TODO...
AppVeyor build 1.0.446 for commit bf0f5132149269d4f8fa7d897bba64806e6887ea is now complete. Found 1 potential spelling error(s). Preview:
content/02.intro.md:8:TODO...
AppVeyor build 1.0.447 for commit bf0f5132149269d4f8fa7d897bba64806e6887ea is now complete. Found 1 potential spelling error(s). Preview:
content/02.intro.md:8:TODO...
Thank you for the review @allaway! In many cases I committed your changes directly, but I also attempted to make edits in the spirit of your review in some cases.
AppVeyor build 1.0.448 for commit 3bb33c935b8ab2ae4cedb71219efabdc01d6e9a3 is now complete. Found 4 potential spelling error(s). Preview:
content/02.intro.md:6:TODO
content/02.intro.md:9:responder
content/02.intro.md:9:responder
content/02.intro.md:16:TODO...
AppVeyor build 1.0.450 Found 4 potential spelling error(s). Preview:
content/02.intro.md:6:TODO
content/02.intro.md:9:responder
content/02.intro.md:9:responder
content/02.intro.md:16:TODO...
@allaway your approval is necessary for merging this request since you had requested changes :)
Purpose
Starts to address #173. I'm not totally happy with this yet, but wanted to get more 👀 on it so folks can see the general direction I am going in.
Quoting https://github.com/jaybee84/ml-in-rd/issues/173#issuecomment-887619198, as that's what I thought we should do when I re-read this for the first time in a while:
I think I've covered a lot of these in this draft, but I also don't know if folks agreed with these takes!
Directions for reviewers
Manuscript checklist
Unless otherwise noted above, this PR will be considered ready for review when all four items have been checked.