Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

upgrade aws-sam-translator#13490

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
alexrashed merged 2 commits intomainfromupgrade-dependencies
Dec 10, 2025
Merged

upgrade aws-sam-translator#13490

alexrashed merged 2 commits intomainfromupgrade-dependencies
Dec 10, 2025

Conversation

@bentsku
Copy link
Contributor

@bentskubentsku commentedDec 9, 2025•
edited
Loading

Motivation

Following#13483, it seemed we downgradedaws-sam-translator back to1.103.0, but as@cloutierMat found out, this has rolled back support for Lambda Managed Instances:https://github.com/aws/serverless-application-model/releases/tag/v1.104.0

This was pinned in Moto due to Python 3.14 issues, which do not concern us yet:getmoto/moto#9521

So this PR is force pinning it at least the latest version as of today.

Changes

  • pinwas-sam-translator to at least1.105.0 and regenerate the requirements

Tests

Related

@bentskubentsku self-assigned thisDec 9, 2025
@bentskubentsku added semver: patchNon-breaking changes which can be included in patch releases docs: skipPull request does not require documentation changes notes: skipPull request does not have to be mentioned in the release notes labelsDec 9, 2025
@github-actions
Copy link

github-actionsbot commentedDec 9, 2025•
edited
Loading

S3 Image Test Results (AMD64 / ARM64)

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

Results for commit743c4bc.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actionsbot commentedDec 9, 2025•
edited
Loading

Test Results - Preflight, Unit

23 000 tests  ±0   21 158 ✅ ±0   6m 28s ⏱️ +6s
     1 suites ±0    1 842 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit743c4bc. ± Comparison against base commit1224e92.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actionsbot commentedDec 9, 2025•
edited
Loading

Test Results (amd64) - Acceptance

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

Results for commit743c4bc. ± Comparison against base commit1224e92.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

Test Results - Alternative Providers

588 tests   331 ✅  17m 15s ⏱️
  1 suites  257 💤
  1 files      0 ❌

Results for commiteab341f.

@github-actions
Copy link

github-actionsbot commentedDec 9, 2025•
edited
Loading

Test Results (amd64) - Integration, Bootstrap

    5 files      5 suites   2h 32m 45s ⏱️
5 510 tests 4 954 ✅ 556 💤 0 ❌
5 516 runs  4 954 ✅ 562 💤 0 ❌

Results for commit743c4bc.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 57m 8s ⏱️ +20s
5 130 tests ±0  4 734 ✅ ±0  396 💤 ±0  0 ❌ ±0 
5 132 runs  ±0  4 734 ✅ ±0  398 💤 ±0  0 ❌ ±0 

Results for commit743c4bc. ± Comparison against base commit1224e92.

@bentskubentsku marked this pull request as ready for reviewDecember 9, 2025 23:38
@bentskubentsku added the dependenciesPull requests that update a dependency file labelDec 9, 2025
Copy link
Member

@alexrashedalexrashed left a comment•
edited
Loading

Choose a reason for hiding this comment

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

Thanks a lot for jumping on this! 💯 The min dependency seems like a good workaround, and it seems that the transitive conflict is not an issue. 🚀 🦸🏽

@alexrashedalexrashed merged commitfe9daa5 intomainDec 10, 2025
92 of 101 checks passed
@alexrashedalexrashed deleted the upgrade-dependencies branchDecember 10, 2025 08:22
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@alexrashedalexrashedalexrashed approved these changes

@silv-iosilv-ioAwaiting requested review from silv-iosilv-io is a code owner

@k-a-ilk-a-ilAwaiting requested review from k-a-ilk-a-il is a code owner

@sannya-singalsannya-singalAwaiting requested review from sannya-singalsannya-singal is a code owner

Assignees

@bentskubentsku

Labels

dependenciesPull requests that update a dependency filedocs: skipPull request does not require documentation changesnotes: skipPull request does not have to be mentioned in the release notessemver: patchNon-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.

3 participants

@bentsku@alexrashed
[8]ページ先頭
©2009-2025 Movatter.jp