Support Python 3.13 and update deps

[dhirschfeld]

All tests pass¹ with no explicit numpy dependency.

Resolves #509
Closes #504 - (changes have been merged here)

Footnotes

1. Except the broken DCO check ↩

[dhirschfeld]

It looks like your DCO workflow might be broken?

[dbaxa]

cc @rcypher-databricks @yunbodeng-db @andrefurlan-db @jackyhu-db @benc-db for review 🙏 .

[samikshya-db]

Thanks for raising this, @dhirschfeld. For passing the DCO test, we need to have signed off by ... in the github commit message - as mentioned in https://github.com/databricks/databricks-sql-python/blob/main/CONTRIBUTING.md#sign-your-work

DCO tests in GitHub Actions ensure code contributions are legally sound. We are implementing stricter DCO checks. Currently, DCO compliance is not enforced as a pre-merge requirement.

Sample PR where DCO passes : https://github.com/databricks/databricks-sql-python/pull/461/commits.

[dbaxa]

Thanks for raising this, @dhirschfeld. For passing the DCO test, we need to have signed off by ... in the github commit message - as mentioned in https://github.com/databricks/databricks-sql-python/blob/main/CONTRIBUTING.md#sign-your-work

DCO tests in GitHub Actions ensure code contributions are legally sound. We are implementing stricter DCO checks. Currently, DCO compliance is not enforced as a pre-merge requirement.

Sample PR where DCO passes : https://github.com/databricks/databricks-sql-python/pull/461/commits.

@samikshya-db - I had sign offs in my commit messages and my PR (#504) still was red for DCO. The check seems to be failing because of permissions issues no?

Run tisonkun/[email protected]
Error: Although you appear to have the correct authorization credentials, the `databricks` organization has an IP allow list enabled, and your IP address is not permitted to access this resource.

& the comment fails due to a different perms issue it seems.

     },
    data: {
      message: 'Resource not accessible by integration',
      documentation_url: 'https://docs.github.com/rest/issues/comments#create-an-issue-comment',
      status: '403'
    }
  },
  request: {
    method: 'POST',
    url: 'https://api.github.com/repos/databricks/databricks-sql-python/issues/504/comments',
    headers: {
      accept: 'application/vnd.github.v3+json',
      'user-agent': 'actions/github-script octokit-core.js/3.6.0 Node.js/20.18.0 (linux; x64)',
      authorization: 'token [REDACTED]',
      'content-type': 'application/json; charset=utf-8'
    },
    body: '{"body":"Thanks for your contribution! To satisfy the DCO policy in our   [contributing guide](https://github.com/databricks/databricks-sql-python/blob/main/CONTRIBUTING.md)   every commit message must include a sign-off message. One or more of your commits is missing this message.   You can reword previous commit messages with an interactive rebase (`git rebase -i main`)."}',
    request: { agent: [Agent], hook: [Function: bound bound register] }
  }
}

[dhirschfeld]

As @dbaxa says, all commits have been signed off. The DCO check is broken because the necessary IP address is being blocked.

[samikshya-db]

DCO sign-off should be part of every commit for the DCO GH action to pass (the merge commit too). The message that you posted is actually a red-herring. I agree that this GH action should be more developer friendly :/ Created : #519 to address this.

Thanks for the contribution!

[dhirschfeld]

Thanks for the merge @samikshya-db!

If anyone here has commit rights in databricks-sqlalchemy I have a similar PR over there to allow installing into a modern environment:

• Remove upper-bound pin on pyarrow databricks-sqlalchemy#20