Issue Details (XML | Word | Printable)

Key: CONTINUUM-2028
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Emmanuel Venisse
Reporter: Wendy Smoak
Votes: 2
Watchers: 2
Operations

If you were logged in you would be able to see more operations.
Continuum

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

Created: 30/Dec/08 04:54 PM   Updated: 08/Feb/09 04:08 PM   Resolved: 04/Feb/09 03:55 PM
Component/s: None
Affects Version/s: 1.2.2
Fix Version/s: 1.3.2 (Beta)

Time Tracking:
Not Specified

Environment: Continuum 1.2.2, Subversion
Issue Links:
Related

Complexity: Intermediate


 Description  « Hide

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>


Wendy Smoak added a comment - 30/Dec/08 05:16 PM

While I'd love to see this fixed, I don't want to load up 1.3.1 with more than absolutely necessary.

This seems to be a larger problem than this one config element – see CONTINUUM1633, CONTINUUM-1503, CONTINUUM-1504, CONTINUUM-1505 which all point to different release plugin config being ignored.


Olivier Lamy added a comment - 30/Dec/08 05:22 PM - edited

Ok.
We can try to fix that for 1.2.3
Will you have time to test this ? (because testing the release through UI need times).


Wendy Smoak added a comment - 30/Dec/08 05:27 PM

Yes I have time to test it. If you've got a fix in mind already, go for it – on trunk!

I simply meant I didn't want this to block a 1.3.1 release once we have the database upgrade issue sorted out. (Fix for 1.3.x instead of 1.3.1, that's all.)


Julien HENRY added a comment - 30/Jan/09 08:51 AM

It would be great to be able to give the scm comment prefix in the Continuum UI when doing the release. I also have a pre-commit hook in my organisation but the issue tracker is different for each release so it is not a good solution for me to store it in the pom.


Wendy Smoak added a comment - 30/Jan/09 09:02 AM

I opened CONTINUUM-2054 for the further enhancement of configuring this through the Web UI.

Even if the issue key changes for every release, you can still configure it in the pom prior to executing the release process.


Julien HENRY added a comment - 30/Jan/09 09:21 AM

Thanks for opening the enhancement request.

The main advantage of releasing through Continuum Web UI is it is not needed to checkout the code source (can be done from a non developer computer). If I have to edit the pom before releasing, I have to checkout/commit so it reduces Continuum usefulness.


Emmanuel Venisse added a comment - 04/Feb/09 03:55 PM

Done in r.740906


Wendy Smoak added a comment - 06/Feb/09 09:42 AM

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.


Emmanuel Venisse added a comment - 07/Feb/09 04:04 AM

It works fine.


Wendy Smoak added a comment - 08/Feb/09 09:06 AM

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


Emmanuel Venisse added a comment - 08/Feb/09 04:08 PM

Yes probably.