Closed cwilso closed 1 year ago
Note that this is superceded by #111 .
should I bring my comments that still apply to #111?
@dontcallmedom I am SO so sorry. I forgot to incorporate your comments here (and this is exactly how I wanted commentary). I've done so now - read through your comments above, and you can see the disposition.
thanks a lot! I've marked as resolved the comments that have been integrated, and will try to do another pass with comments/issues for the remaining ones
(Nitpick: you can make this a draft PR so it's clear it's not to be merged, the merge affordances don't show, etc.)