Skip to content

fix: fix error for each helper with block params and strict mode #1820 - #1855

Merged
jaylinski merged 1 commit into
handlebars-lang:4.xfrom
arturmakaranka:fix/block-params-for-each-helper-in-strict-mode
May 25, 2022
Merged

jaylinski merged 1 commit into
handlebars-lang:4.xfrom
arturmakaranka:fix/block-params-for-each-helper-in-strict-mode

Conversation

@arturmakaranka

Copy link
Copy Markdown
Contributor

Fix for #each helper with params in strict mode. By the same pr of @jaylinski https://github.com/handlebars-lang/handlebars.js/pull/1821/files

@arturmakaranka arturmakaranka changed the title wip: fix error for each helper with block params and strict mode fix: fix error for each helper with block params and strict mode May 5, 2022
@arturmakaranka arturmakaranka changed the title fix: fix error for each helper with block params and strict mode fix: fix error for each helper with block params and strict mode #1820 May 5, 2022
@jaylinski jaylinski self-assigned this May 6, 2022

@jaylinski jaylinski left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please run npm run format and commit the changes in order to fix the linting error.

@arturmakaranka
arturmakaranka requested a review from jaylinski May 10, 2022 06:35
@arturmakaranka
arturmakaranka changed the base branch from 4.x to master May 24, 2022 09:13
@arturmakaranka
arturmakaranka changed the base branch from master to 4.x May 24, 2022 09:13
@arturmakaranka

Copy link
Copy Markdown
Contributor Author

@jaylinski can you please merge this fix? or how often it happens? As I understand I can't impact ot this process/

@jaylinski
jaylinski force-pushed the fix/block-params-for-each-helper-in-strict-mode branch from 1845af8 to 86a9615 Compare May 25, 2022 20:12
@jaylinski
jaylinski merged commit 30dbf04 into handlebars-lang:4.x May 25, 2022
@jaylinski

Copy link
Copy Markdown
Member

ℹ️ This fix was added to master in b14ff4a.

@jaylinski jaylinski mentioned this pull request May 25, 2022
3 tasks done
@jaylinski

jaylinski commented May 25, 2022 •

Copy link
Copy Markdown
Member

@arturmakaranka Thank you for the fix! Very much appreciated.

I'll try to release the fix as soon as possible, but it may take some time.

andersk added a commit to andersk/zulip that referenced this pull request Mar 22, 2023
The fix at handlebars-lang/handlebars.js#1855
is not released yet.

Signed-off-by: Anders Kaseorg <anders@zulip.com>
andersk added a commit to andersk/zulip that referenced this pull request Mar 22, 2023
The fix at handlebars-lang/handlebars.js#1855
is not released yet.

Signed-off-by: Anders Kaseorg <anders@zulip.com>
timabbott pushed a commit to timabbott/zulip that referenced this pull request Mar 23, 2023
The fix at handlebars-lang/handlebars.js#1855
is not released yet.

Signed-off-by: Anders Kaseorg <anders@zulip.com>
sbansal1999 pushed a commit to sbansal1999/zulip that referenced this pull request Apr 1, 2023
The fix at handlebars-lang/handlebars.js#1855
is not released yet.

Signed-off-by: Anders Kaseorg <anders@zulip.com>
plugyawn pushed a commit to plugyawn/armada that referenced this pull request Jul 30, 2026
The fix at handlebars-lang/handlebars.js#1855
is not released yet.

Signed-off-by: Anders Kaseorg <anders@zulip.com>
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.

2 participants