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

Upgrade pinned Python dependencies - #10435

Merged
alexrashed merged 4 commits into
masterfrom
upgrade-dependencies
Mar 12, 2024
Merged

Upgrade pinned Python dependencies#10435
alexrashed merged 4 commits into
masterfrom
upgrade-dependencies

Conversation

@localstack-bot

@localstack-bot localstack-bot commented Mar 12, 2024

Copy link
Copy Markdown
Contributor

This PR upgrades all the pinned Python dependencies.

@localstack-bot localstack-bot added area: dependencies Pull requests that update a dependency file semver: patch Non-breaking changes which can be included in patch releases labels Mar 12, 2024
@github-actions

github-actions Bot commented Mar 12, 2024 •
edited
Loading

Copy link
Copy Markdown

S3 Image Test Results (AMD64 / ARM64)

  2 files    2 suites   3m 18s ⏱️
393 tests 342 ✅  51 💤 0 ❌
786 runs  684 ✅ 102 💤 0 ❌

Results for commit c4db5c1.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Mar 12, 2024 •
edited
Loading

Copy link
Copy Markdown

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 28m 53s ⏱️ - 1m 28s
2 705 tests +1  2 449 ✅ +1  256 💤 ±0  0 ❌ ±0 
2 707 runs  +1  2 449 ✅ +1  258 💤 ±0  0 ❌ ±0 

Results for commit c4db5c1. ± Comparison against base commit 55eff22.

♻️ This comment has been updated with latest results.

@alexrashed
fix ruff 0.3.1 debugpy addition to T100
eced7fd 
ruff 0.3.1 added linting rules to alert on the usage of debugpy
with astral-sh/ruff#10194.
This commit excludes these lines because it's used for a specific
feature (instead of debugging code leaking into the codebase).
@alexrashed
add upper limit for rolo
c7ef8c4 
rolo==0.4.0 contains some breaking changes addressed in
#10428.
This commit pins rolo to versions lower than 0.4.0.
This limit should be removed (or at least increased) in #10428.
@alexrashed alexrashed mentioned this pull request Mar 12, 2024
Merged
3 tasks
@thrau

thrau commented Mar 12, 2024

Copy link
Copy Markdown
Contributor

Instead of pinning rolo can we just fix the import error? Should be just the one ASGIWebsocket import and one test that uses it that need to be removed

@coveralls

coveralls commented Mar 12, 2024 •
edited
Loading

Copy link
Copy Markdown

Coverage Status

coverage: 85.683% (-0.006%) from 85.689%
when pulling c4db5c1 on upgrade-dependencies
into 55eff22 on master.

@localstack-bot
localstack-bot requested a review from thrau as a code owner March 12, 2024 09:29
@alexrashed
alexrashed merged commit 5d27f04 into master Mar 12, 2024
@alexrashed
alexrashed deleted the upgrade-dependencies branch March 12, 2024 10:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@silv-io silv-io Awaiting requested review from silv-io
@thrau thrau Awaiting requested review from thrau
1 more reviewer
@alexrashed alexrashed alexrashed approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

area: dependencies Pull requests that update a dependency file 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.

4 participants

@localstack-bot @thrau @coveralls @alexrashed