ibis
ibis copied to clipboard
chore: adjust <Backend>.do_connect() be kw or positional only, add callouts to url connection method
resolves https://github.com/ibis-project/ibis/issues/10901
This not only adds better docs, but it also is slightly breaking
by mkaing many args kwarg-only. So if someone was doing ibis.bigquery.connect("my-project-id", "my-dataset-id") then that would be broken. For most backends, there are so many connection params, I would expect most devs to pass them by kwarg already, but I bet this will affect some people.
This is also potentially slightly breaking for duckdb and sqlite by making the database path positional-only. I'm not as sure about if this is a good idea.
Let me know, I can adjust the scope of this as you want.
ACTION NEEDED
Ibis follows the Conventional Commits specification for release automation.
The PR title and description are used as the merge commit message.
Please update your PR title and description to match the specification.