This is a breaking change. But we can add a new set of methods called close_by_ref

Hmm, yeah, I agree this is probably technically a breaking change. However, this would only break when using the fully qualified syntax, e.g. DatabaseConnection::close(connection). This case also isn't specifically mentioned in https://doc.rust-lang.org/cargo/reference/semver.html. So, it may be okay to introduce this change IMO, but that's just my two cents and I'll leave it up to the maintainers to decide.

@spencewenski technically, the reference doesn't mention changing a type of existing parameter either, but it would clearly be a breaking change :)
so I agree with close_by_ref

@tyt2y3 I've changed the implementation as you advised

🎉 Released In 1.1.7 🎉

Thank you everyone for the contribution!
This feature is now available in the latest release. Now is a good time to upgrade!
Your participation is what makes us unique; your adoption is what drives us forward.
You can support SeaQL 🌊 by starring our repos, sharing our libraries and becoming a sponsor ⭐.