Closed kcranston closed 7 years ago
:+1:
Thanks @kcranston. 👍
Since there are enough unresolved comments, it doesn't seem like this can be merged. So, I'm going to work in this branch to address the comments and the other ones others have mentioned in the emails.
Let me know if there's an objection to me doing the updates in this branch.
Note that a comment thread on an individual line does go away when that section gets updated with new PRs.
+1
I addressed the issues here, but I also made quite a few other updates. So, I'm going to merge this PR, and then put in a new one.
First pass at reviewer comments. See also issue #170 and #169