Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.6
    • Fix Version/s: 2.8
    • Component/s: jira
    • Labels:
      None
    • Testcase included:
      yes
    • Patch Submitted:
      Yes
    • Number of attachments :
      2

      Description

      We use the JIRA report in a reactor build, using Maven's reporting plugin inheritance to have child projects generate their own reports. However, some of the projects - such as the reactor build POMs themselves - have no real relevant JIRA components to be used in the report (they're just build aggregates). Right now, we just short-circuit the report in those cases by providing it a phony component ID, but it'd be nice if we could just skip it altogether, since our current approach still generates a JIRA report, but with an error message.

      I've attached a patch with the proposed changes.

      1. skip-jira.patch
        2 kB
        Joshua Hyde
      2. skip-jira-username.patch
        2 kB
        Joshua Hyde

        Activity

        Hide
        Joshua Hyde added a comment -

        Just realized that the patch lists my computer's username, not my JIRA username, as the author on the test case. In case the @author tag survives the review process, I'm submitting a patch with the correct username so that traceability is a tad easier.

        Show
        Joshua Hyde added a comment - Just realized that the patch lists my computer's username, not my JIRA username, as the author on the test case. In case the @author tag survives the review process, I'm submitting a patch with the correct username so that traceability is a tad easier.
        Hide
        Dennis Lundberg added a comment -

        Patch applied in r1382996 with some modifications. The plugin has moved to using annotations for mojo parameters, so I adjusted the patch accordingly.

        Thank you for the patch!

        Show
        Dennis Lundberg added a comment - Patch applied in r1382996 with some modifications. The plugin has moved to using annotations for mojo parameters, so I adjusted the patch accordingly. Thank you for the patch!

          People

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

            Dates

            • Created:
              Updated:
              Resolved: