Details

      Description

      debug(), info() and error() are not intended for messages: they are only intended for log level displaying from slf4j provider implementation: having these methods in MessageBuilder is confusing (and hard to explain)
      moving them to another interface would make things more clear

        Issue Links

          Activity

          Hide
          hudson Hudson added a comment -

          SUCCESS: Integrated in Jenkins build maven-shared #2866 (See https://builds.apache.org/job/maven-shared/2866/)
          MSHARED-587 typos (hboutemy: http://svn.apache.org/viewvc/?view=rev&rev=1759176)

          • (edit) maven-shared-utils/pom.xml
          • (edit) maven-shared-utils/src/main/java/org/apache/maven/shared/utils/logging/MessageUtils.java
            MSHARED-587 extracted LoggerLevelRenderer interface from MessageBuilder (hboutemy: http://svn.apache.org/viewvc/?view=rev&rev=1759175)
          • (edit) maven-shared-utils/src/main/java/org/apache/maven/shared/utils/logging/AnsiMessageBuilder.java
          • (add) maven-shared-utils/src/main/java/org/apache/maven/shared/utils/logging/LoggerLevelRenderer.java
          • (edit) maven-shared-utils/src/main/java/org/apache/maven/shared/utils/logging/MessageBuilder.java
          • (edit) maven-shared-utils/src/main/java/org/apache/maven/shared/utils/logging/MessageUtils.java
          • (edit) maven-shared-utils/src/main/java/org/apache/maven/shared/utils/logging/PlainMessageBuilder.java
          • (edit) maven-shared-utils/src/main/java/org/apache/maven/shared/utils/logging/Style.java
          • (edit) maven-shared-utils/src/main/java/org/apache/maven/shared/utils/logging/package-info.java
          • (edit) maven-shared-utils/src/test/java/org/apache/maven/shared/utils/logging/AnsiMessageBuilderTest.java
          Show
          hudson Hudson added a comment - SUCCESS: Integrated in Jenkins build maven-shared #2866 (See https://builds.apache.org/job/maven-shared/2866/ ) MSHARED-587 typos (hboutemy: http://svn.apache.org/viewvc/?view=rev&rev=1759176 ) (edit) maven-shared-utils/pom.xml (edit) maven-shared-utils/src/main/java/org/apache/maven/shared/utils/logging/MessageUtils.java MSHARED-587 extracted LoggerLevelRenderer interface from MessageBuilder (hboutemy: http://svn.apache.org/viewvc/?view=rev&rev=1759175 ) (edit) maven-shared-utils/src/main/java/org/apache/maven/shared/utils/logging/AnsiMessageBuilder.java (add) maven-shared-utils/src/main/java/org/apache/maven/shared/utils/logging/LoggerLevelRenderer.java (edit) maven-shared-utils/src/main/java/org/apache/maven/shared/utils/logging/MessageBuilder.java (edit) maven-shared-utils/src/main/java/org/apache/maven/shared/utils/logging/MessageUtils.java (edit) maven-shared-utils/src/main/java/org/apache/maven/shared/utils/logging/PlainMessageBuilder.java (edit) maven-shared-utils/src/main/java/org/apache/maven/shared/utils/logging/Style.java (edit) maven-shared-utils/src/main/java/org/apache/maven/shared/utils/logging/package-info.java (edit) maven-shared-utils/src/test/java/org/apache/maven/shared/utils/logging/AnsiMessageBuilderTest.java
          Hide
          hudson Hudson added a comment -

          SUCCESS: Integrated in Jenkins build maven-shared #2875 (See https://builds.apache.org/job/maven-shared/2875/)
          MSHARED-587 improved documentation (hboutemy: http://svn.apache.org/viewvc/?view=rev&rev=1760149)

          • (edit) maven-shared-utils/src/main/java/org/apache/maven/shared/utils/logging/package-info.java
          Show
          hudson Hudson added a comment - SUCCESS: Integrated in Jenkins build maven-shared #2875 (See https://builds.apache.org/job/maven-shared/2875/ ) MSHARED-587 improved documentation (hboutemy: http://svn.apache.org/viewvc/?view=rev&rev=1760149 ) (edit) maven-shared-utils/src/main/java/org/apache/maven/shared/utils/logging/package-info.java

            People

            • Assignee:
              hboutemy Hervé Boutemy
              Reporter:
              hboutemy Hervé Boutemy
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development