Code review comment for lp:~justizin/zope.copypastemove/lp_98385

Revision history for this message
Tres Seaver (tseaver) wrote :

I will merge with a couple of tweaks:

- We really prefer not to add any dependencies, even testing dependencies, on
  the 'zope.app' packages from outside that namespace. As it turns out, the
  'zope.container' package provides all these features anywa, which allows me
  to use the new test with a trivial tweak.

- The 'renameItem' methods should *all* return the newName: I'm updating the
  one for ordered containers to do so.

« Back to merge proposal