Merge lp:~stevenk/launchpad/db-recipe-description-really-this-time into lp:launchpad/db-devel

Proposed by Steve Kowalik
Status: Merged
Approved by: Robert Collins
Approved revision: no longer in the source branch.
Merged at revision: 10247
Proposed branch: lp:~stevenk/launchpad/db-recipe-description-really-this-time
Merge into: lp:launchpad/db-devel
Diff against target: 12 lines (+1/-1)
1 file modified
lib/lp/code/interfaces/sourcepackagerecipe.py (+1/-1)
To merge this branch: bzr merge lp:~stevenk/launchpad/db-recipe-description-really-this-time
Reviewer Review Type Date Requested Status
Robert Collins (community) Approve
William Grant code* Approve
Review via email: mp+51682@code.launchpad.net

Commit message

[r=lifeless,wgrant][bug=713518] SourcePackageRecipe.description is now really optional.

Description of the change

Re-add required to description for SourcePackageRecipe, but set it to false, since it defaults to True.

To post a comment you must log in.
Revision history for this message
William Grant (wgrant) wrote :

Fail.

review: Approve (code*)
Revision history for this message
Robert Collins (lifeless) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'lib/lp/code/interfaces/sourcepackagerecipe.py'
2--- lib/lp/code/interfaces/sourcepackagerecipe.py 2011-02-28 14:03:09 +0000
3+++ lib/lp/code/interfaces/sourcepackagerecipe.py 2011-03-01 03:14:09 +0000
4@@ -221,7 +221,7 @@
5 "be unique for the given owner.")))
6
7 description = exported(Description(
8- title=_('Description'),
9+ title=_('Description'), required=False,
10 description=_('A short description of the recipe.')))
11
12 date_last_modified = exported(

Subscribers

People subscribed via source and target branches

to status/vote changes: