Continuum

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

Vote (1)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: