Skip to content

Upgrade pinned Python dependencies#10435

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

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

Conversation

@localstack-bot
Copy link
Contributor

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

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
Copy link

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

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
Copy link

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

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
Copy link
Member

thrau commented Mar 12, 2024

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
Copy link

coveralls commented Mar 12, 2024 •
edited
Loading

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 join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alexrashed alexrashed alexrashed approved these changes

@silv-io silv-io Awaiting requested review from silv-io

@thrau thrau Awaiting requested review from thrau

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