Skip to content

chore(json-logic): Add skipped broken unit test for missing json-logic reduce support #98

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

Merged
merged 3 commits into from
Mar 10, 2025

Conversation

brennj
Copy link
Collaborator

@brennj brennj commented Nov 27, 2024

This was a bug brought up internally when trying to create a schema similar to the test case. This is because of a false assumption on my side where I didn't need to account for "temp variables" that are indeed supported in json logic for cases like reduce.

@brennj brennj requested review from ollyd and sandrina-p November 27, 2024 21:05
@brennj brennj force-pushed the ensure-reduce-is-supported branch from f544251 to d0e402b Compare February 4, 2025 15:39
@brennj brennj changed the title fix: reduce for json logic was not working Add skipped broken unit test for missing json-logic reduce support Feb 4, 2025
@brennj
Copy link
Collaborator Author

brennj commented Feb 4, 2025

rebased with main @dragidavid / @thien-remote with the move to next I'll leave this in your capable hands to ensure this is eventually moved into there. Feel free to merge/disregard this PR!

@sandrina-p sandrina-p force-pushed the ensure-reduce-is-supported branch from d0e402b to 67d98ec Compare March 6, 2025 14:02
@sandrina-p sandrina-p changed the title Add skipped broken unit test for missing json-logic reduce support chore(json-logic): Add skipped broken unit test for missing json-logic reduce support Mar 6, 2025
@sandrina-p sandrina-p merged commit 8dc801b into main Mar 10, 2025
3 checks passed
@sandrina-p sandrina-p deleted the ensure-reduce-is-supported branch March 10, 2025 10:57
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