Skip to content
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

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

Conversation

arturmakaranka
Copy link
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
Copy link
Member

@jaylinski jaylinski left a comment

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 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
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
Member

ℹ️ This fix was added to master in b14ff4a.

@jaylinski
Copy link
Member

jaylinski commented May 25, 2022

@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
andersk added a commit to andersk/zulip that referenced this pull request Mar 22, 2023
timabbott pushed a commit to timabbott/zulip that referenced this pull request Mar 23, 2023
sbansal1999 pushed a commit to sbansal1999/zulip that referenced this pull request Apr 1, 2023
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