Skip to content

make "and" and "or" accept any number of parameters#687

Merged
sunng87 merged 1 commit into
sunng87:masterfrom
lovasoa:or_and_multiple_params
Dec 23, 2024
Merged

make "and" and "or" accept any number of parameters#687
sunng87 merged 1 commit into
sunng87:masterfrom
lovasoa:or_and_multiple_params

Conversation

@lovasoa

@lovasoa lovasoa commented Dec 20, 2024

Copy link
Copy Markdown
Contributor

Writing ifs with multiple conditions used to be very painful, and not very performant

One had to write {{#if (and (cond1 (and cond2 cond3)))}}

One can now write {{#if (and cond1 cond2 cond3)}}

Writing ifs with multiple conditions used to be very painful, and not very performant

One had to write {{#if (and (cond1 (and cond2 cond3)))}}

One can now write {{#if (and cond1 cond2 cond3)}}
@lovasoa lovasoa force-pushed the or_and_multiple_params branch from 0b932ea to caf7fd6 Compare December 20, 2024 22:43
@coveralls

coveralls commented Dec 20, 2024

Copy link
Copy Markdown

Coverage Status

coverage: 82.062% (+0.04%) from 82.024%
when pulling caf7fd6 on lovasoa:or_and_multiple_params
into 190bc3b on sunng87:master.

@sunng87 sunng87 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thank you!

@sunng87 sunng87 merged commit 0df66fa into sunng87:master Dec 23, 2024
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