You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
dreamorosi opened this issue
Apr 16, 2024
· 2 comments
· Fixed by #2367
Assignees
Labels
completedThis item is complete and has been merged/shippedinternalPRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.)jmespathThis item relates to the JMESPath Utility
Customer using the JMESPath utility can customize the way expressions are parsed by specifying options, for example they can extend the functions available.
Currently the option and corresponding type for this feature is called ParsingOption which is overly generic.
We should rename it to clarify what it does and where it comes from.
Why is this needed?
Because the utility is being consumed by other packages like Idempotency and because there's another upcoming utility called Parser.
Namespacing the type by using something like JMESPathParsingOptions would immediately make it clear where the option comes from and what it's for.
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.
The text was updated successfully, but these errors were encountered:
dreamorosi
added
internal
PRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.)
confirmed
The scope is clear, ready for implementation
jmespath
This item relates to the JMESPath Utility
labels
Apr 16, 2024
completedThis item is complete and has been merged/shippedinternalPRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.)jmespathThis item relates to the JMESPath Utility
Summary
Customer using the JMESPath utility can customize the way expressions are parsed by specifying options, for example they can extend the functions available.
Currently the option and corresponding type for this feature is called
ParsingOption
which is overly generic.We should rename it to clarify what it does and where it comes from.
Why is this needed?
Because the utility is being consumed by other packages like Idempotency and because there's another upcoming utility called Parser.
Namespacing the type by using something like
JMESPathParsingOptions
would immediately make it clear where the option comes from and what it's for.Which area does this relate to?
JMESPath
Solution
No response
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: