Skip to content

Conversation

@etspaceman
Copy link
Contributor

@dfangl dfangl added the semver: patch Non-breaking changes which can be included in patch releases label Dec 18, 2023
@alexrashed alexrashed requested a review from dfangl December 19, 2023 08:59
Copy link
Member

@alexrashed alexrashed left a comment

Choose a reason for hiding this comment

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

Thanks as always for pro-actively updating the version after implementing the fix!
The failing tests are caused by:

  • The fork not having the permissions to execute secrets (to execute the pro tests).
  • An issue with the upgrade of dynamodblocal, which is addressed in #9910.

From my point of view this PR is looking good and could be merged, but I have limited availability today to take a closer look. So I'll leave it to @dfangl to review / approve / merge. Thanks!

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

Labels

semver: patch Non-breaking changes which can be included in patch releases

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants