Skip to content

Update the minimum version of databricks-labs-pytester to 0.7.2 #408

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

asnare
Copy link
Contributor

@asnare asnare commented May 14, 2025

The just released version of pytester includes a fix for a breaking change that was introduced by Databricks SDK 0.51: older versions of pytester will fail to import if the new Databricks SDK is present.

This PR updates the test dependencies for this project to require that version or later. This is mainly an informational change: it doesn't affect normal installs of lsql, nor does it affect fresh installs of this project for development because the latest version will be picked up.

Versions before this trigger import errors with Databricks SDK 0.51 or later.
@asnare asnare self-assigned this May 14, 2025
@asnare asnare added the dependencies Pull requests that update a dependency file label May 14, 2025
Copy link

github-actions bot commented May 14, 2025

✅ 36/36 passed, 4 skipped, 32m31s total

Running from acceptance #501

@asnare asnare added this to UCX May 14, 2025
@asnare asnare moved this to Ready for Review in UCX May 14, 2025
@asnare asnare changed the title Update the minimum version of databricks-labs-pytester to 0.7.2 Update the minimum version of databricks-labs-pytester to 0.7.2 May 14, 2025
@asnare asnare moved this from Ready for Review to Blocked/Hold in UCX May 14, 2025
@asnare asnare added the pr/do-not-merge This PR is not ready to be merged label May 14, 2025
@asnare
Copy link
Contributor Author

asnare commented May 14, 2025

Looks like some more work will be needed to untangle things. All good now.

@asnare asnare marked this pull request as draft May 14, 2025 09:39
@asnare asnare marked this pull request as ready for review May 15, 2025 17:29
@asnare asnare moved this from Blocked/Hold to Ready for Review in UCX May 15, 2025
@asnare asnare removed the pr/do-not-merge This PR is not ready to be merged label May 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
Status: Ready for Review
Development

Successfully merging this pull request may close these issues.

1 participant