necouchman commented on pull request #604: URL: https://github.com/apache/guacamole-client/pull/604#issuecomment-824313142
Just at a glance, your commit messages should be a little more descriptive. You have 9 commit messages that say "Update ko.json", and then the final one is relatively similar (Add multiple ko.json files). Commit messages should describe at a high level what the commit is doing, and if all 9 of those commit messages are really updating the same ko.json file, they should be squashed together. For reference: https://www.codelord.net/2015/03/16/bad-commit-messages-hall-of-shame/ -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
