ibis icon indicating copy to clipboard operation
ibis copied to clipboard

chore: adjust <Backend>.do_connect() be kw or positional only, add callouts to url connection method

Open NickCrews opened this issue 8 months ago • 1 comments

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.

NickCrews avatar Mar 12 '25 18:03 NickCrews

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.

github-actions[bot] avatar Mar 12 '25 18:03 github-actions[bot]