Any more early opinions on https://github.com/voxpupuli/puppet-ldapquery/pull/47 ? A separate dispatch to the existing function name seemed sensible at first, but now I'm not too sure. I could just make it a new

ldapquery::search

function and leave the existing one 100% as is?? Either way, getting the API 'right' so we're not changing it again soon is important.