yeah, I hate that naming convention but I'll remove the _ from the name. On Thu, Jun 18, 2015 at 5:50 AM, karuturi <g...@git.apache.org> wrote: > Github user karuturi commented on the pull request: > > https://github.com/apache/cloudstack/pull/467#issuecomment-113027041 > > @DaanHoogland looks like you split the commit :) (the combined one was > also ok. ) > > I see some checkstyle errors in the jenkins build > [INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ > cloud-utils --- > [INFO] Starting audit... > > /home/jenkins/jenkins-slave/workspace/cloudstack-pull-requests/utils/src/com/cloud/utils/StringUtils.java:36:28: > Name '_acsCharset' must match pattern '^(s_)?[a-z][a-zA-Z0-9]*$'. > Audit done. > > > > --- > If your project is set up for it, you can reply to this email and have your > reply appear on GitHub as well. If your project does not have this feature > enabled and wishes so, or if the feature is enabled but not working, please > contact infrastructure at infrastruct...@apache.org or file a JIRA ticket > with INFRA. > ---
-- Daan
