vishesh92 commented on PR #9873: URL: https://github.com/apache/cloudstack/pull/9873#issuecomment-2567438705
> you are going to hate me @vishesh92 , but can we use capitals for acronyms please? I marked it in a few places but it is all over the place. It is more in line with other discussions on the same subject (see amongst others #9132) otherwise please involve yourself in the discussion on the topic. @DaanHoogland Making this change in this PR would be difficult due to the size of the changes. It would be better to tackle this separately once we have reached a consensus in #9132. Also, I have raised this PR against 4.20. There might be some merge conflicts while forward merging 4.19 to 4.20 but those were going to come anyway due to the logger variable rename in 4.20 branch. -- 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]
