Continuum

Ability to configure the scm comment prefix through the web UI for release prepare

Details

  • Type: New Feature New Feature
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.3.1 (Alpha)
  • Fix Version/s: 1.3.2 (Beta)
  • Component/s: Release, Web - UI
  • Labels:
    None
  • Complexity:
    Intermediate
  • Patch Submitted:
    Yes
  • Number of attachments :
    1

Description

Further enhancement to CONTINUUM-2028

"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." -Julien HENRY

Issue Links

is related to

Improvement - An improvement or enhancement to an existing feature or task. CONTINUUM-2028 Continuum release prepare should use maven-release-plugin scmCommentPrefix configuration in pom

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Wendy Smoak made changes -
Field Original Value New Value
Link This issue is related to CONTINUUM-2028 [ CONTINUUM-2028 ]
Wendy Smoak made changes -
Fix Version/s 1.3.2 [ 14902 ]
Component/s Release [ 12699 ]
Assignee Wendy Smoak [ wsmoak ]
Patch Submitted [Yes]
Hide
Permalink
Wendy Smoak added a comment -

Attaching patch of work in progress; still need to externalize the label on the web page and figure out how to test it.

Show
Wendy Smoak added a comment - Attaching patch of work in progress; still need to externalize the label on the web page and figure out how to test it.
Wendy Smoak made changes -
Attachment CONTINUUM-2054-wsmoak-20090201-1058.diff [ 39703 ]
Hide
Permalink
Wendy Smoak added a comment -

Fixed in r740864.

There is a new field for the scm comment prefix on the 'release prepare' page, and the value is passed through to the release process.

If no value is provided, it will use the default of "[maven-release-manager] ".

A possible enhancement would be to append a trailing space if none is present.

This could use an automated test, however the current tests look for contents in files, and I don't see how to test that the commit had the correct comment. As long as we're passing the value through correctly, the code doing the actual work is over in Maven [Release|Shared|SCM].

Show
Wendy Smoak added a comment - Fixed in r740864. There is a new field for the scm comment prefix on the 'release prepare' page, and the value is passed through to the release process. If no value is provided, it will use the default of " [maven-release-manager] ". A possible enhancement would be to append a trailing space if none is present. This could use an automated test, however the current tests look for contents in files, and I don't see how to test that the commit had the correct comment. As long as we're passing the value through correctly, the code doing the actual work is over in Maven [Release|Shared|SCM] .
Wendy Smoak made changes -
Resolution Fixed [ 1 ]
Status Open [ 1 ] Closed [ 6 ]

People

  • Assignee:
    Wendy Smoak
    Reporter:
    Wendy Smoak
Vote (1)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: