← Back to team overview

launchpad-reviewers team mailing list archive

Re: [Merge] lp:~cjwatson/launchpad/zope.tales-upgrade into lp:launchpad

 

I think in practice it doesn't matter very much for properties, and the existing codebase seems to be inconsistent on this.  However, I interpret that documentation and the zope.schema._bootstrapfields.Field code as saying that a required field may never take a value equal to the declared missing_value, which would be None here, and thus required=True would technically be correct (although it will never make a difference because Field.validate will always pass either way).  Does that make sense to you?
-- 
https://code.launchpad.net/~cjwatson/launchpad/zope.tales-upgrade/+merge/247030
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.


References