On 5/12/17 9:11 AM, Mandy Chung wrote:
Minor comment: 95 private final List<String> COMMON_ARGS; This is an instance field and you can use lower case as the convention. 238 return !s.equals(""); You can consider using String::isEmpty.
Thanks, Mandy. I will make these changes before pushing. -Brent