Move dill to the runtime dependencies #13508

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.

Sign up for GitHub

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

Jump to bottom

Merged

giograno merged 2 commits into main from dill-runtime-dep

Dec 23, 2025

Member

giograno commented Dec 11, 2025 •

edited

Loading

Motivation

We used to have dill as a base dependency since it was needed by the CLI for state serialization.
Nowadays, serialization is done completely in the runtime; therefore, this is not strictly necessary.

Changes

Moving dill in the runtime dependencies.

Addresses PNX-566

Tests

Related

giograno self-assigned this

giograno added semver: patch docs: skip notes: skip labels

github-actions bot commented Dec 11, 2025 •

edited

Loading

S3 Image Test Results (AMD64 / ARM64)

2 files 2 suites 8m 23s ⏱️
552 tests 500 ✅ 52 💤 0 ❌
1 104 runs 1 000 ✅ 104 💤 0 ❌

Results for commit 01b140b.

♻️ This comment has been updated with latest results.

github-actions bot commented Dec 11, 2025 •

edited

Loading

Test Results - Preflight, Unit

23 046 tests ±0 21 201 ✅ ±0 6m 16s ⏱️ +13s
1 suites ±0 1 845 💤 ±0
1 files ±0 0 ❌ ±0

Results for commit 01b140b. ± Comparison against base commit 960e535.

♻️ This comment has been updated with latest results.

github-actions bot commented Dec 11, 2025 •

edited

Loading

Test Results (amd64) - Acceptance

7 tests ±0 5 ✅ ±0 2m 58s ⏱️ +9s
1 suites ±0 2 💤 ±0
1 files ±0 0 ❌ ±0

Results for commit 01b140b. ± Comparison against base commit 960e535.

♻️ This comment has been updated with latest results.

github-actions bot commented Dec 11, 2025 •

edited

Loading

Test Results (amd64) - Integration, Bootstrap

5 files 5 suites 2h 35m 54s ⏱️
5 561 tests 5 001 ✅ 560 💤 0 ❌
5 567 runs 5 001 ✅ 566 💤 0 ❌

Results for commit 01b140b.

♻️ This comment has been updated with latest results.

giograno force-pushed the dill-runtime-dep branch 2 times, most recently from e63f4b6 to 4a6db1c Compare

December 12, 2025 16:55

github-actions bot commented Dec 12, 2025 •

edited

Loading

LocalStack Community integration with Pro

2 files ±0 2 suites ±0 1h 56m 30s ⏱️ + 1m 25s
5 154 tests ±0 4 757 ✅ ±0 397 💤 ±0 0 ❌ ±0
5 156 runs ±0 4 757 ✅ ±0 399 💤 ±0 0 ❌ ±0

Results for commit 01b140b. ± Comparison against base commit 960e535.

♻️ This comment has been updated with latest results.

giograno added 2 commits

December 23, 2025 15:11


          dill as runtime dependency

757c1d2


          update dill requirements

01b140b

giograno force-pushed the dill-runtime-dep branch from 4a6db1c to 01b140b Compare

December 23, 2025 14:11

giograno marked this pull request as ready for review

December 23, 2025 14:52

giograno requested review from alexrashed, bentsku, k-a-il and silv-io as code owners

December 23, 2025 14:52

silv-io approved these changes

View reviewed changes

Member

silv-io left a comment

Nice cleanup 🤩

giograno merged commit 449854d into main

70 of 71 checks passed

giograno deleted the dill-runtime-dep branch

December 23, 2025 17:24

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

silv-io silv-io approved these changes

alexrashed Awaiting requested review from alexrashed alexrashed is a code owner

k-a-il Awaiting requested review from k-a-il k-a-il is a code owner

bentsku Awaiting requested review from bentsku bentsku is a code owner

Labels

docs: skip notes: skip semver: patch