Continuum
  1. Continuum
  2. CONTINUUM-2028

Continuum release prepare should use maven-release-plugin scmCommentPrefix configuration in pom

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.2.2
    • Fix Version/s: 1.3.2 (Beta)
    • Component/s: None
    • Labels:
      None
    • Environment:
      Continuum 1.2.2, Subversion
    • Complexity:
      Intermediate
    • Number of attachments :
      0

      Description

      I have a Subversion repository with a pre-commit hook that requires a valid issue tracker id in every commit message.

      The default commit message for release prepare is "[maven-release-manager] prepare for next development iteration" which is not acceptable.

      I tried configuring the scmCommentPrefix for the release plugin in the pom, but the commit message still came out as [maven-release-manager] and was rejected.

        <build>
          <plugins>
            <plugin>
              <artifactId>maven-release-plugin</artifactId>
              <configuration>
                <scmCommentPrefix>[ABCD1234]</scmCommentPrefix>
              </configuration>
            </plugin>
          </plugins>
        </build>
      

        Issue Links

          Activity

          Hide
          Emmanuel Venisse added a comment -

          Done in r.740906

          Show
          Emmanuel Venisse added a comment - Done in r.740906
          Hide
          Wendy Smoak added a comment -

          This doesn't seem to be working in r740908

          with this in the pom:

          <plugin>
          <artifactId>maven-release-plugin</artifactId>
          <configuration>
          <autoVersionSubmodules>true</autoVersionSubmodules>
          <scmCommentPrefix>[HELLO-455] </scmCommentPrefix>
          </configuration>
          </plugin>

          the commit message was:

          Author: wsmoak
          Date: 2009-02-06 09:35:17 -0600 (Fri, 06 Feb 2009)
          New Revision: 1182

          Modified:
          hello/trunk/pom.xml
          Log:
          [maven-release-manager] prepare release hello-1.1.6

          I also noticed the SCM Comment Prefix on the release prepare page didn't pre-populate with the value in the pom, so it may be related to CONTINUUM-2054.

          I'll modify one of the examples in the sandbox and record a Selenium test to double check.

          Show
          Wendy Smoak added a comment - This doesn't seem to be working in r740908 with this in the pom: <plugin> <artifactId>maven-release-plugin</artifactId> <configuration> <autoVersionSubmodules>true</autoVersionSubmodules> <scmCommentPrefix> [HELLO-455] </scmCommentPrefix> </configuration> </plugin> the commit message was: Author: wsmoak Date: 2009-02-06 09:35:17 -0600 (Fri, 06 Feb 2009) New Revision: 1182 Modified: hello/trunk/pom.xml Log: [maven-release-manager] prepare release hello-1.1.6 I also noticed the SCM Comment Prefix on the release prepare page didn't pre-populate with the value in the pom, so it may be related to CONTINUUM-2054 . I'll modify one of the examples in the sandbox and record a Selenium test to double check.
          Hide
          Emmanuel Venisse added a comment -

          It works fine.

          Show
          Emmanuel Venisse added a comment - It works fine.
          Hide
          Wendy Smoak added a comment -

          It works fine for me at r741756. Thanks for taking a look, Emmanuel. Maybe you fixed it by adding the missing 'get' method?

          Show
          Wendy Smoak added a comment - It works fine for me at r741756. Thanks for taking a look, Emmanuel. Maybe you fixed it by adding the missing 'get' method?
          Hide
          Emmanuel Venisse added a comment -

          Yes probably.

          Show
          Emmanuel Venisse added a comment - Yes probably.

            People

            • Assignee:
              Emmanuel Venisse
              Reporter:
              Wendy Smoak
            • Votes:
              2 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: