Add tiledb main branch to CI matrix #697
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sgillies
commented
Jan 23, 2025
| @@ -37,6 +37,7 @@ jobs: | |||
| run: pre-commit run -a -v | |||
|
|
|||
| run-tests: | |||
| name: "${{ matrix.tiledb-version-spec }}:${{ matrix.python-version }}:${{ matrix.dependencies }}:${{ matrix.os }}" | |||
There was a problem hiding this comment.
So that the tiledb version appears first in the limited width left sidebar on, for example, https://github.com/TileDB-Inc/TileDB-Cloud-Py/actions/runs/12937321533.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
With this PR, we now test tiledb-cloud against 1) the minimum supported version of tiledb(-py), 2) the most recent supported release on PyPI, and 3) the main branch of the tiledb-py project.
In the first case, we
pip install tiledb==0.30and, after,pip install .[tests]determines that 0.30.0 meets the requirements and uses it.In the second case, we
pip install tiledb, get 0.33.2, and thenpip install .[tests]determines that 0.33.2 is excluded, and then replaces it with a new install of 0.33.0 from PyPI.In the third case, we install tiledb-py's main branch from GitHub, and
pip install .[tests]determines that it meets the requirements.The current test failures are due to timeouts on the server side. I'll rerun and they'll pass.