You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@cloudstack.apache.org by GitBox <gi...@apache.org> on 2021/09/01 09:25:41 UTC

[GitHub] [cloudstack] GutoVeronezi commented on pull request #5386: Replace com.google.common.base.Strings for org.apache.commons.lang3.StringUtils

GutoVeronezi commented on pull request #5386:
URL: https://github.com/apache/cloudstack/pull/5386#issuecomment-909463414


   > I like the readability of Strings.isNullOrEmpty than isEmpty, maybe refactor everything to use cloud-utils's StringsUtils that wraps whatever favourite utility dependency you want to propose. Discuss on dev@ as Daan suggests.
   
   I agree that some methods can be more intuitive than others, however the point is to standardize what we're using. Soon I will start a discussion on the dev list, as @DaanHoogland suggested.
   


-- 
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: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org