GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Move dill to the runtime dependencies by giograno · Pull Request #13508 · localstack/localstack

This repository was archived by the owner on Mar 23, 2026. It is now read-only.

Conversation

@giograno

@giograno giograno commented Dec 11, 2025 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member

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

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
bentsku and silv-io reacted with rocket emoji
All reactions
  • 🚀 2 reactions
@giograno giograno self-assigned this Dec 11, 2025
@giograno giograno added semver: patch Non-breaking changes which can be included in patch releases docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes labels Dec 11, 2025
@github-actions

github-actions Bot commented Dec 11, 2025 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

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.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@github-actions

github-actions Bot commented Dec 11, 2025 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

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.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@github-actions

github-actions Bot commented Dec 11, 2025 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

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.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@github-actions

github-actions Bot commented Dec 11, 2025 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

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.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@giograno
giograno force-pushed the dill-runtime-dep branch 2 times, most recently from e63f4b6 to 4a6db1c Compare December 12, 2025 16:55
@github-actions

github-actions Bot commented Dec 12, 2025 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

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.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
giograno added 2 commits December 23, 2025 15:11
@giograno
dill as runtime dependency
757c1d2
@giograno
update dill requirements
01b140b
@giograno
giograno force-pushed the dill-runtime-dep branch from 4a6db1c to 01b140b Compare December 23, 2025 14:11
@giograno
giograno marked this pull request as ready for review December 23, 2025 14:52
@giograno
giograno requested review from alexrashed, bentsku, k-a-il and silv-io as code owners December 23, 2025 14:52
silv-io
silv-io approved these changes Dec 23, 2025
View reviewed changes

@silv-io silv-io left a comment

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Nice cleanup 🤩

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@giograno
giograno merged commit 449854d into main Dec 23, 2025
70 of 71 checks passed

Uh oh!

There was an error while loading. Please reload this page.
@giograno
giograno deleted the dill-runtime-dep branch December 23, 2025 17:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@alexrashed alexrashed Awaiting requested review from alexrashed alexrashed is a code owner
@k-a-il k-a-il Awaiting requested review from k-a-il k-a-il is a code owner
@bentsku bentsku Awaiting requested review from bentsku
1 more reviewer
@silv-io silv-io silv-io approved these changes
Reviewers whose approvals may not affect merge requirements

Uh oh!

There was an error while loading. Please reload this page.

Assignees

@giograno giograno

Labels

docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes semver: patch Non-breaking changes which can be included in patch releases

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@giograno @silv-io
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.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.