Continuum
  1. Continuum
  2. CONTINUUM-409

Email notifications could still include build stats when includeBuildResult is false

    Details

    • Type: Wish Wish
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.1-beta-3
    • Fix Version/s: 1.1-beta-4
    • Component/s: Notifier - Mail
    • Labels:
      None
    • Complexity:
      Intermediate
    • Patch Submitted:
      Yes
    • Number of attachments :
      1

      Description

      I really like the "Build statistics" and "Changes" sections continuum creates in the emails and definitely want them in every email. It's really just the section that contains the output generated from the build itself that I don't want as that will consistently be several megs of text for our project.

      Basically, I assumed 'includeBuildResults=false' would just exclude out everything after these lines

      ****************************************************************************
      Output:
      ****************************************************************************

      I really don't care what the option is called as long as I can get an email as described.

      Maybe you want to do something like this

      <includeBuildStatistics>true</includeBuildStatistics>
      <includeChanges>true</includeChanges>
      <includeBuildOutput>false</includeBuildOutput>

        Activity

        Hide
        Tomislav Stojcevich added a comment -

        Here is a patch that allows a new attribute <includeBuildSummary> that can be used to control the summary information. It's not as robust as allowing individual sections to be turned on or off but I think this will at least allow those who want the summary but not the build result in the email to get by until the more robust CONTINUUM-310 gets included.

        Show
        Tomislav Stojcevich added a comment - Here is a patch that allows a new attribute <includeBuildSummary> that can be used to control the summary information. It's not as robust as allowing individual sections to be turned on or off but I think this will at least allow those who want the summary but not the build result in the email to get by until the more robust CONTINUUM-310 gets included.
        Hide
        Emmanuel Venisse added a comment -

        Applied with includeBuildSummary in application.xml.
        Thanks.

        Show
        Emmanuel Venisse added a comment - Applied with includeBuildSummary in application.xml. Thanks.

          People

          • Assignee:
            Emmanuel Venisse
            Reporter:
            David Blevins
          • Votes:
            1 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: