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

Allow testing of earliest/latest dependencies. #230

Closed
wants to merge 3 commits into from

Conversation

seberg
Copy link
Contributor

@seberg seberg commented Jul 15, 2024

Mostly identical to gh-228, which allows the testing of oldest/latest dependencies. What oldest/latest dependencies means exactly has to be set by each project in their dependencies.yaml file for the test env.

I am interested in it, because I would like to start testing with NumPy 2, but don't like the idea of stopping testing with 1.x at the same time. But e.g. cudf testing with older pandas or cuGraph with older networkx would be good as well.

Modifications from draft PR:

  • I renamed it to oldest, as suggested by Matthew.
  • Noticed that the name is different between wheel and conda workflows, so modified both to include all matrix information.

(Marking as draft until I have linked PRs that show it working also for wheels, but Bradley did really all the work here, and it seems to work well.)

@seberg seberg marked this pull request as ready for review July 15, 2024 12:41
@seberg seberg requested a review from a team as a code owner July 15, 2024 12:41
@seberg seberg requested review from bdice and removed request for a team July 15, 2024 12:41
@seberg
Copy link
Contributor Author

seberg commented Jul 15, 2024

Hmmm, marking as non-draft either way. I don't think I can use this workflow reasonably in other PRs unless the branch lives directly in the shared workflows repo.

@bdice
Copy link
Contributor

bdice commented Jul 15, 2024

@seberg Correct — your fork won’t suffice for testing. I would recommend that you close this PR, request write access on this repo from ops, and push directly to my PR.

@seberg seberg closed this Jul 15, 2024
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.

2 participants