-
Notifications
You must be signed in to change notification settings - Fork 171
Closed
Labels
completedThis item is complete and has been merged/shippedThis item is complete and has been merged/shippedinternalPRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.)PRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.)parserThis item relates to the Parser UtilityThis item relates to the Parser Utility
Description
Summary
In #3442 we missed to add @aws-sdk/util-dynamodb
package to peerDependenciesMeta
as option, which now pulls the dependency for some package managers like pnpm
.
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
- This request meets Powertools for AWS Lambda (TypeScript) Tenets
- Should this be considered in other Powertools for AWS Lambda languages? i.e. Python, Java, and .NET
Future readers
Please react with 👍 and your use case to help us understand customer demand.
Metadata
Metadata
Assignees
Labels
completedThis item is complete and has been merged/shippedThis item is complete and has been merged/shippedinternalPRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.)PRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.)parserThis item relates to the Parser UtilityThis item relates to the Parser Utility
Type
Projects
Status
Shipped