<p>Excellent - this is exactly what I had in mind. Some notes:</p>

<p>"Resultset specifies..." sounds odd, given that the callsite reported will be the search() containing the attr in question. Also when possible we try to stick to a single sentence for warnings/exceptions. A better wording (which is also more in line with other deprecation notices) is "Resultset attribute 'foo' has been deprecated - use 'bar' instead".</p>

<p>Your first carp_unique has an extra quote mark at the end.</p>

<p>Please also add a line abot these to Changes (user-visible changes)</p>

<p>I noticed the email you used in the author list is different than what you used to make the commit. Intentional or oversight or...?</p>

<p>Cheers and thanks!</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br>Reply to this email directly or <a href='https://github.com/dbsrgits/dbix-class/pull/19#issuecomment-15116419'>view it on GitHub</a>.<img src='https://github.com/notifications/beacon/4W3BWTo7EIi_7vx28Xzl_n0sJ_S_KeuDlL9kpefWYDxy1pGuNNLJl_U1_ppAOgYs.gif' height='1' width='1'></p>