[Recursion Concept]: Fixed Typos & Formatting#4209
Conversation
Yrahcaz7
left a comment
There was a problem hiding this comment.
Mostly looks good, I just have a few suggestions and nitpicks.
Co-authored-by: Yrahcaz <74512479+Yrahcaz7@users.noreply.github.com>
Co-authored-by: Yrahcaz <74512479+Yrahcaz7@users.noreply.github.com>
Co-authored-by: Yrahcaz <74512479+Yrahcaz7@users.noreply.github.com>
Yrahcaz7
left a comment
There was a problem hiding this comment.
Looks good! There's one typo, but it isn't actually visible to readers, so it's really just a minor nitpick.
|
@BethanyG, if you didn't know, there is a "Batch Changes" button for applying suggested changes all at once (in one commit), rather applying them separately (a commit for each). It doesn't matter too much as the commits are squashed on merge, but I thought it might be helpful to know. |
Co-authored-by: Yrahcaz <74512479+Yrahcaz7@users.noreply.github.com>
Co-authored-by: Yrahcaz <74512479+Yrahcaz7@users.noreply.github.com>
Thanks. 😄 I did know that, but the web interface where you batch changes (the detailed code review pane) now collapses comments by default, and so when there are only a few, it is far easier to identify them and commit them from the main screen. But yeah - it gets noisy and fiddly. Some changes MS has made are great. Others ... not so much. 😉 |
|
LMK when this is ready to merge. 😄 |
Per Issue 4165.