On Fri, Dec 12, 2008 at 02:24:09PM +0100, Martijn Faassen wrote: > Hey, > > Christian Zagrodnick wrote: > [snip] > > That's good. One thing which is not good is that you deprecated the use > > of ITerms from zope.app.form. I'd just leave the reference/import there > > like we did with ISite in zope.app.component. > > Why is such a deprecation warning bad? Wouldn't this encourage people to > update their code?
One issue is that it's impossible to write code that's "deprecation warning free" and works across multiple versions of dependencies. That forces people to become accustomed to seeing and ignoring deprecation warnings. > > Regards, > > Martijn > > > > _______________________________________________ > Zope-Dev maillist - Zope-Dev@zope.org > http://mail.zope.org/mailman/listinfo/zope-dev > ** No cross posts or HTML encoding! ** > (Related lists - > http://mail.zope.org/mailman/listinfo/zope-announce > http://mail.zope.org/mailman/listinfo/zope ) -- Brian Sutherland _______________________________________________ Zope-Dev maillist - Zope-Dev@zope.org http://mail.zope.org/mailman/listinfo/zope-dev ** No cross posts or HTML encoding! ** (Related lists - http://mail.zope.org/mailman/listinfo/zope-announce http://mail.zope.org/mailman/listinfo/zope )
