Skip to content

[SFN] [IntrinsicFunctions] base support for JsonMerge and States.Hash, boolean literals support, snap test#7671

Merged
MEPalma merged 60 commits intomasterfrom
MEP-sfn-intrinsic-json-merge
Feb 24, 2023
Merged

[SFN] [IntrinsicFunctions] base support for JsonMerge and States.Hash, boolean literals support, snap test#7671
MEPalma merged 60 commits intomasterfrom
MEP-sfn-intrinsic-json-merge

Conversation

@MEPalma
Copy link
Contributor

@MEPalma MEPalma commented Feb 13, 2023

  • Adds base support for the missing Json Manipulation intrinsic function 'States.JsonMerge'.
  • Adds base support for all States hashing intrinsic functions
  • Snapshot test for this new function
  • Adds support for parsing boolean literal arguments in intrinsic functions functions calls

@MEPalma MEPalma temporarily deployed to localstack-ext-tests February 14, 2023 19:38 — with GitHub Actions Inactive
@MEPalma MEPalma temporarily deployed to localstack-ext-tests February 14, 2023 20:52 — with GitHub Actions Inactive
@MEPalma MEPalma changed the title [SFN] base support for JsonMerge intrinsic function, boolean literals support, snap test [SFN] [IntrinsicFunctions] base support for JsonMerge and States.Hash, boolean literals support, snap test Feb 14, 2023
@MEPalma MEPalma temporarily deployed to localstack-ext-tests February 18, 2023 15:51 — with GitHub Actions Inactive
@coveralls
Copy link

coveralls commented Feb 18, 2023

Coverage Status

Coverage: 85.056% (-0.1%) from 85.158% when pulling 5e7e46a on MEP-sfn-intrinsic-json-merge into 8eaabbd on master.

@MEPalma MEPalma temporarily deployed to localstack-ext-tests February 22, 2023 11:11 — with GitHub Actions Inactive
@MEPalma MEPalma temporarily deployed to localstack-ext-tests February 23, 2023 11:04 — with GitHub Actions Inactive
@MEPalma MEPalma temporarily deployed to localstack-ext-tests February 23, 2023 12:54 — with GitHub Actions Inactive
Copy link
Member

@dominikschubert dominikschubert left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for also already including the nits from the last reviews! 💪

@MEPalma MEPalma merged commit 49904db into master Feb 24, 2023
@alexrashed alexrashed deleted the MEP-sfn-intrinsic-json-merge branch March 27, 2023 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants