Maven Changelog Plugin
  1. Maven Changelog Plugin
  2. MCHANGELOG-70

Support a URL filter that enables JIRA/bugzilla/whatever IDs quoted in SCM message to be mapped to real URLs

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.1
    • Fix Version/s: 2.2
    • Labels:
      None

      Description

      Eclipse Mylyn, plus other good behaviours (see JIRA Commit Acceptance Plugin) make it necessary to quote a task upon check-in. These task IDs can easily be mapped to URLs using simple regex pattern rules that can be defined in the plugin config.

      1. MCHANGELOG-70,71,72.patch
        11 kB
        skaze
      2. MCHANGELOG-70,71,72.patch
        11 kB
        skaze
      3. MCHANGELOG-70.patch
        5 kB
        skaze
      4. MCHANGELOG-70.patch
        5 kB
        skaze
      5. MCHANGELOG-70.patch
        5 kB
        skaze

        Activity

        skaze created issue -
        Hide
        skaze added a comment -

        Patch provides two new options:

            /**
             * A pattern used to identify 'issue tracker' IDs such as those used by JIRA, 
             * Bugzilla and alike in the SCM commit messages. Any matched patterns
             * are replaced with <code>ticketLinkUrl<code> URL. The default
             * value is a JIRA-style ticket identification pattern.
             *
             * @parameter expression="${ticketLinkRegexPattern}" default-value="[a-zA-Z]{2,}-\d+"
             * @required
             */
            private String ticketLinkRegexPattern;
        
            /**
             * The issue tracker URL used in replacing any matched <code>ticketLinkRegexPattern</code>
             * found in the SCM commit messages. The default is URL is the codehaus JIRA
             * URL. If %TICKET% if found in the URL it is replaced with the matched ticket ID,
             * otherwise the matched ticket ID is appended to the URL.
             *
             * @parameter expression="${ticketLinkUrl}" default-value="http://jira.codehaus.org/browse/%TICKET%"
             * @required
             */
            private String ticketLinkUrl;
        

        Which, if defined, will replace any matching 'ticket identifiers' found in the commit messages with links to the ticket system based on the 'ticketLinkUrl' template string.

        Patch generated against trunk, revision 560535.

        Combined patch for MCHANGELOG-70, MCHANGELOG-71 and MCHANGELOG-72 to follow.

        Show
        skaze added a comment - Patch provides two new options: /** * A pattern used to identify 'issue tracker' IDs such as those used by JIRA, * Bugzilla and alike in the SCM commit messages. Any matched patterns * are replaced with <code>ticketLinkUrl<code> URL. The default * value is a JIRA-style ticket identification pattern. * * @parameter expression= "${ticketLinkRegexPattern}" default -value= "[a-zA-Z]{2,}-\d+" * @required */ private String ticketLinkRegexPattern; /** * The issue tracker URL used in replacing any matched <code>ticketLinkRegexPattern</code> * found in the SCM commit messages. The default is URL is the codehaus JIRA * URL. If %TICKET% if found in the URL it is replaced with the matched ticket ID, * otherwise the matched ticket ID is appended to the URL. * * @parameter expression= "${ticketLinkUrl}" default -value= "http: //jira.codehaus.org/browse/%TICKET%" * @required */ private String ticketLinkUrl; Which, if defined, will replace any matching 'ticket identifiers' found in the commit messages with links to the ticket system based on the 'ticketLinkUrl' template string. Patch generated against trunk, revision 560535. Combined patch for MCHANGELOG-70 , MCHANGELOG-71 and MCHANGELOG-72 to follow.
        skaze made changes -
        Field Original Value New Value
        Attachment MCHANGELOG-70.patch [ 28666 ]
        Hide
        skaze added a comment -

        Mistake in the 'ticketLinkRegexPattern' default value - an extra escape character was required, i.e.

             * @parameter expression="${ticketLinkRegexPattern}" default-value="[a-zA-Z]{2,}-\d+"
        

        should have been:

             * @parameter expression="${ticketLinkRegexPattern}" default-value="[a-zA-Z]{2,}-\\d+"
        
        Show
        skaze added a comment - Mistake in the 'ticketLinkRegexPattern' default value - an extra escape character was required, i.e. * @parameter expression= "${ticketLinkRegexPattern}" default -value= "[a-zA-Z]{2,}-\d+" should have been: * @parameter expression= "${ticketLinkRegexPattern}" default -value= "[a-zA-Z]{2,}-\\d+"
        skaze made changes -
        Attachment MCHANGELOG-70.patch [ 28667 ]
        Hide
        skaze added a comment -

        Combined patch for MCHANGELOG-70, MCHANGELOG-71 and MCHANGELOG-72 - much easier to apply once the individuals have been reviewed and accepted.

        Show
        skaze added a comment - Combined patch for MCHANGELOG-70 , MCHANGELOG-71 and MCHANGELOG-72 - much easier to apply once the individuals have been reviewed and accepted.
        skaze made changes -
        Attachment MCHANGELOG-70,71,72.patch [ 28668 ]
        Hide
        skaze added a comment -

        fixes unit tests failures from NPEs due to a lack of parameter injected values.

        Show
        skaze added a comment - fixes unit tests failures from NPEs due to a lack of parameter injected values.
        skaze made changes -
        Attachment MCHANGELOG-70.patch [ 28675 ]
        Hide
        skaze added a comment -

        fixes NPEs in unit tests due to a lack of injected parameters.

        Show
        skaze added a comment - fixes NPEs in unit tests due to a lack of injected parameters.
        skaze made changes -
        Attachment MCHANGELOG-70,71,72.patch [ 28677 ]
        Hide
        Dennis Lundberg added a comment -

        Fixed in r939060.
        Thanks!

        I changed the names of the parameters to look similar to the ones used in Maven Changes Plugin.

        Show
        Dennis Lundberg added a comment - Fixed in r939060 . Thanks! I changed the names of the parameters to look similar to the ones used in Maven Changes Plugin.
        Dennis Lundberg made changes -
        Assignee Dennis Lundberg [ dennislundberg ]
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Closed [ 6 ]
        Fix Version/s 2.2 [ 13634 ]
        Mark Thomas made changes -
        Project Import Sun Apr 05 09:12:33 UTC 2015 [ 1428225153301 ]
        Mark Thomas made changes -
        Workflow jira [ 12717771 ] Default workflow, editable Closed status [ 12749137 ]
        Mark Thomas made changes -
        Project Import Sun Apr 05 22:37:13 UTC 2015 [ 1428273433745 ]
        Mark Thomas made changes -
        Workflow jira [ 12955210 ] Default workflow, editable Closed status [ 12991853 ]
        Mark Thomas made changes -
        Assignee dennislundberg [ dennislundberg ] Dennis Lundberg [ dennisl@apache.org ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Closed Closed
        1007d 36m 1 Dennis Lundberg 28/Apr/10 17:54

          People

          • Assignee:
            Dennis Lundberg
            Reporter:
            skaze
          • Votes:
            2 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development