work-items-tracker

Merge lp://staging/~mabac/launchpad-work-items-tracker/fix-none-whiteboard into lp://staging/~linaro-automation/launchpad-work-items-tracker/linaro

  1. fix-none-whiteboard
  2. Merge into linaro
Proposed by Mattias Backman
Status: Merged
Merged at revision: 334
Proposed branch: lp://staging/~mabac/launchpad-work-items-tracker/fix-none-whiteboard
Merge into: lp://staging/~linaro-automation/launchpad-work-items-tracker/linaro
Diff against target: 12 lines (+2/-0)
1 file modified
collect (+2/-0)
To merge this branch: bzr merge lp://staging/~mabac/launchpad-work-items-tracker/fix-none-whiteboard
Reviewer Review Type Date Requested Status
Guilherme Salgado (community) Approve
Review via email: mp+99294@code.staging.launchpad.net

Description of the change

Hi,

This branch adds a check for the whiteboard being None before concatenating the workitems_text prior to parsing.

It seems that some (new?) blueprints have a None whiteboard.

Thanks,

Mattias

To post a comment you must log in.
Revision history for this message
Mattias Backman (mabac) wrote :

This is in the middle of a lot of stuff so not trivial to add a test. Let this code base be a lesson to us all. ;)

Revision history for this message
Guilherme Salgado (salgado) wrote :

Looks good to me

review: Approve
Revision history for this message
Mattias Backman (mabac) wrote :

On Mon, Mar 26, 2012 at 2:29 PM, Guilherme Salgado
<email address hidden> wrote:
> Review: Approve
>
> Looks good to me

Thanks. Merged to linaro-staging and it looks good.

> --
> https://code.launchpad.net/~mabac/launchpad-work-items-tracker/fix-none-whiteboard/+merge/99294
> You are the owner of lp:~mabac/launchpad-work-items-tracker/fix-none-whiteboard.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches