Continuum
  1. Continuum
  2. CONTINUUM-2054

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

    Details

    • Type: New Feature New Feature
    • Status: 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
    • Flags:
      Patch

      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

          Activity

          Wendy Smoak created issue -
          Wendy Smoak made changes -
          Field Original Value New Value
          Link This issue is related to CONTINUUM-2028 [ CONTINUUM-2028 ]
          Wendy Smoak made changes -
          Assignee Wendy Smoak [ wsmoak ]
          Patch Submitted [Yes]
          Component/s Release [ 12699 ]
          Fix Version/s 1.3.2 [ 14902 ]
          Hide
          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
          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 ]
          Mark Thomas made changes -
          Project Import Sun Apr 05 08:36:01 UTC 2015 [ 1428222961749 ]
          Mark Thomas made changes -
          Workflow jira [ 12710706 ] Default workflow, editable Closed status [ 12738748 ]
          Mark Thomas made changes -
          Patch Submitted Yes [ 10763 ]
          Flags Patch [ 10430 ]
          Mark Thomas made changes -
          Project Import Sun Apr 05 21:12:18 UTC 2015 [ 1428268338676 ]
          Mark Thomas made changes -
          Workflow jira [ 12946412 ] Default workflow, editable Closed status [ 12984420 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Closed Closed
          5d 5h 37m 1 Wendy Smoak 04/Feb/09 14:37

            People

            • Assignee:
              Wendy Smoak
              Reporter:
              Wendy Smoak
            • Votes:
              1 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development