-
Notifications
You must be signed in to change notification settings - Fork 153
Maintenance: remove aws-sdk/util-dynamodb dependency in favour of commons package #3492
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
I think we should add the custom function we added in #3482 and remove the dependency altogether instead. The function is present in the commons package and is already being used by the DynamoDB schema. |
I agree, will adjust the title and description, as well the PR. |
This issue is now closed. Please be mindful that future comments are hard for our team to see. If you need more assistance, please either tag a team member or open a new issue that references this one. If you wish to keep having a conversation with other community members under this issue feel free to do so. |
This is now released under v2.13.1 version! |
Summary
In #3442 we missed to add@aws-sdk/util-dynamodb
package topeerDependenciesMeta
as option, which now pulls the dependency for some package managers likepnpm
.Remove util-dynamodb dependency since we have implemented the unmarshall function in our commons package.
Why is this needed?
Because we have moved the implementation of
unmarshall
function to commons package.Which area does this relate to?
Parser
Solution
Add commons dependency to parser utility.
Acknowledgment
Future readers
Please react with 👍 and your use case to help us understand customer demand.
The text was updated successfully, but these errors were encountered: