Feature request: remove usage of isDate in Metrics utility internals #3461
Assignees
Labels
completed
This item is complete and has been merged/shipped
feature-request
This item refers to a feature request for an existing or new utility
metrics
This item relates to the Metrics Utility
Comments
dreamorosi
added
confirmed
github-project-automation
bot
moved this to Triage
in Powertools for AWS Lambda (TypeScript)
dreamorosi
moved this from Triage
to Working on it
in Powertools for AWS Lambda (TypeScript)
dreamorosi
moved this from Working on it
to Coming soon
in Powertools for AWS Lambda (TypeScript)
|
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. |
github-actions
bot
added
pending-release
|
This is now released under v2.13.0 version! |
github-actions
bot
added
completed
dreamorosi
moved this from Coming soon
to Shipped
in Powertools for AWS Lambda (TypeScript)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Use case
To improve compatibility with LLRT, we could replace the internal usage of the
isDatemethod to a simplerinstanceof Datecheck.While we still maintain our stance of best effort support for LLRT, this change is an extremely low hanging fruit.
Solution/User Experience
The change should be totally transparent for customers and not require any change on their part.
Alternative solutions
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: