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

Drop support for zict <2.1.0 #7456

Merged
merged 1 commit into from
Jan 6, 2023
Merged

Drop support for zict <2.1.0 #7456

merged 1 commit into from
Jan 6, 2023

Conversation

crusaderky
Copy link
Collaborator

@crusaderky crusaderky commented Jan 6, 2023

zict 2.1.0 was released 11 months ago.
This PR cleans up a bunch of dead code.

CI and requirements.txt have already been updated in #7285.

@crusaderky crusaderky self-assigned this Jan 6, 2023
@crusaderky crusaderky changed the title Drop support for zict < 2.1.0 Drop support for zict <2.1.0 Jan 6, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Jan 6, 2023

Unit Test Results

See test report for an extended history of previous test failures. This is useful for diagnosing flaky tests.

       24 files  ±0         24 suites  ±0   9h 54m 10s ⏱️ - 14m 2s
  3 314 tests ±0    3 206 ✔️  - 2     105 💤 ±0  3 +2 
39 072 runs  ±0  37 182 ✔️  - 2  1 886 💤 ±0  4 +2 

For more details on these failures, see this check.

Results for commit bd0a699. ± Comparison against base commit e1f1524.

@mrocklin
Copy link
Member

mrocklin commented Jan 6, 2023

Seems fine to me. Should we change the version in requirements.txt?

@crusaderky
Copy link
Collaborator Author

Seems fine to me. Should we change the version in requirements.txt?

It was already (improperly) updated in #7285

@crusaderky crusaderky merged commit 24ba207 into dask:main Jan 6, 2023
@crusaderky crusaderky deleted the zict_210 branch January 6, 2023 16:48
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