Instead of adding comments inline to the pull request that explain your changes, it would have been nicer to handle them as individual commits, with your coment as commit message.