JoaoJandre commented on PR #9132: URL: https://github.com/apache/cloudstack/pull/9132#issuecomment-3655098825
@jbampton, the PR description states that: "This PR cleans up the code base and standardizes some terms for casing." On line 106 of api/src/main/java/org/apache/cloudstack/api/command/admin/internallb/StartInternalLBVMCmd.java, you changed "Vm Id" to "VM ID". However, on the rest of the changes, you changed "Vm Id" to "VM Id". This inconsistency was brough up over a year ago and still was not changed. I am not scope creeping the PR, I have only asked for a single change, one that keeps the PR consistent. Furthermore, this PR could have been merged over a year ago, but you waited more than year to comment. The change I am asking is simple, and has been the same from the very start. You have no right to complain about the delay on the PR. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
