Tests: Remove obsolete Python version skips - #296
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
florinutz
left a comment
There was a problem hiding this comment.
looks good, and thank you for the contribution!
please also have a look at tests/vector_test.py:147, pyproject.toml:81-85 and docs/dataframe.rst:60-61
2dec488 to
62af008
Compare
|
Replacement PR: #299 During follow-up, an amend in a shallow clone briefly disconnected this source branch, causing GitHub to auto-close the PR. I rebuilt the change from current upstream main on a fresh branch, preserved the reviewed changes, applied the requested follow-ups, and reran validation. Apologies for the disruption. |
Fixes #271
Summary
Validation
uv run poe lintuv run pytest -q tests/bulk_test.py(2 passed, 3 skipped)tests/test_support_pandas.pyagainst CrateDB 6.5.0 (5 passed)tests/integration.pyagainst CrateDB 6.5.0 (6 passed)