1. I'd like to see the MR description/commit message say something about what this MR is for and what it covers. The current message doesn't really tell me much.
2. Some things to fix below. Let me know if any of the suggestions aren't clear. I used the standard ed syntax.
Looking good. Some notes:
1. I'd like to see the MR description/commit message say something about what this MR is for and what it covers. The current message doesn't really tell me much.
2. Some things to fix below. Let me know if any of the suggestions aren't clear. I used the standard ed syntax.