Skip to content
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

Depend on core >= 2.15.2, < 2.16 #17

Closed
wants to merge 2 commits into from
Closed

Conversation

johnkerl
Copy link
Contributor

@johnkerl johnkerl commented Apr 26, 2023

As in single-cell-data/TileDB-SOMA#1296 and single-cell-data/TileDB-SOMA#1303, for the same reasons already narrated there.

Copy link
Member

@Shelnutt2 Shelnutt2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pinning needs to have an upperbound to prevent bringing in 2.16 or newer. Something like tiledb >=2.15.1,<2.16 (taken from https://github.com/conda-forge/tiledb-py-feedstock/blob/f153315c81da2ed5f2d89a968c94892aca4b0bde/recipe/meta.yaml#L35)

@jdblischak
Copy link
Collaborator

Since you are changing the recipe but not the package version, now is when you bump the build number from 0 to 1

@johnkerl
Copy link
Contributor Author

@jdblischak @Shelnutt2 thank you both for teaching me things!

That said, I'm going to hold off b/c TileDB-Py 0.21.3 with core 2.15.2 is just around the corner:
TileDB-Inc/TileDB-Py#1754

@johnkerl johnkerl marked this pull request as draft April 26, 2023 19:56
@johnkerl johnkerl changed the title Depend on core >= 2.15.1 Depend on core >= 2.15.2, < 2.16 Apr 27, 2023
@johnkerl johnkerl force-pushed the kerl/tiledb-py-0.21.2 branch 3 times, most recently from 67db962 to 02ce0ca Compare April 27, 2023 14:43
@johnkerl johnkerl requested a review from Shelnutt2 April 27, 2023 14:44
johnkerl added a commit that referenced this pull request May 2, 2023
@johnkerl
Copy link
Contributor Author

johnkerl commented May 2, 2023

Folding this into #18

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.

None yet

3 participants