launchpad-reviewers team mailing list archive

[Colin Watson <mp+411090@xxxxxxxxxxxxxxxxxx>]

Diff comments:

> diff --git a/lib/lp/services/database/enumcol.py b/lib/lp/services/database/enumcol.py
> index 3360ebe..2ce5d24 100644
> --- a/lib/lp/services/database/enumcol.py
> +++ b/lib/lp/services/database/enumcol.py
> @@ -71,6 +71,8 @@ class DBEnum(SimpleProperty):
>  
>  
>  class DBSchemaEnumCol(sqlobject.PropertyAdapter, DBEnum):
> +    """Deprecated; a SQLObject property representing a DBEnumeratedType."""

Well, it certainly does produce a lot of noise, but perhaps that's a good thing to encourage people to fix it.  I've switched this to a `DeprecationWarning` now.

> +
>      def __init__(self, **kw):
>          try:
>              enum = kw.pop('enum')


-- 
https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/411090
Your team Launchpad code reviewers is requested to review the proposed merge of ~cjwatson/launchpad:enumcol-is-deprecated into launchpad:master.