Skip to content

Update dagster-weaviate lockfile#321

Merged
cmpadden merged 3 commits into
mainfrom
update-dagster-weaviate-lockfile
May 22, 2026
Merged

Update dagster-weaviate lockfile#321
cmpadden merged 3 commits into
mainfrom
update-dagster-weaviate-lockfile

Conversation

@cmpadden

Copy link
Copy Markdown
Contributor

Summary

  • refresh dagster-weaviate uv.lock with latest compatible dependency versions

Testing

  • uv lock --upgrade in libraries/dagster-weaviate

@cmpadden

Copy link
Copy Markdown
Contributor Author

Pulled the failing check logs with gh. The lockfile update pulled in a Weaviate client version that performs the cloud metadata check through sync httpx.Client.send; the test only mocked httpx.AsyncClient.send, so it attempted to resolve the fake cloud hostname. Updated the test to mock both sync and async httpx send paths and share the same response assertion helper. Verified locally with uv run pytest dagster_weaviate_tests/test_resource.py::test_cloud_resource -q and pushed the fix.

@cmpadden

Copy link
Copy Markdown
Contributor Author

Pulled the new failing check logs with gh after the mock fix. The only remaining failure was Ruff formatting on dagster_weaviate_tests/test_resource.py. Ran uv run ruff format in libraries/dagster-weaviate, committed the formatting fix, and pushed it.

@cmpadden cmpadden merged commit 74400ea into main May 22, 2026
7 checks passed
@cmpadden cmpadden deleted the update-dagster-weaviate-lockfile branch May 22, 2026 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant