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

[CLEANUP] Drop support for {{#with}} keyword #1574

Merged
merged 1 commit into from Mar 9, 2024
Merged

[CLEANUP] Drop support for {{#with}} keyword #1574

merged 1 commit into from Mar 9, 2024

Conversation

chancancode
Copy link
Contributor

This was dropped on Ember side a while back, just never cleaned up

Ironically this probably meant when Ember dropped the deprecation/ error, nothing was actually stopping {{#with}} from anymore, so perhaps for a while we have actually re-introduced support for the syntax for some time?

Extracted from #1557

This was dropped on Ember side a while back, just never cleaned up

Ironically this probably meant when Ember dropped the deprecation/
error, nothing was actually stopping `{{#with}}` from anymore, so
perhaps for a while we have actually _re-introduced_ support for
the syntax for some time?

Extracted from #1557
@chancancode
Copy link
Contributor Author

chancancode commented Mar 9, 2024

Ironically this probably meant when Ember dropped the deprecation/ error, nothing was actually stopping {{#with}} from anymore, so perhaps for a while we have actually re-introduced support for the syntax for some time?

Yeah,I think we actually did, just tried in on 5.4 and {{#with}} certainly works there.

Specifically, in here: emberjs/ember.js#19632 / emberjs/ember.js#19638

This is partially why, among other reasons, when removing a deprecated feature, we typically want to swap the deprecate call for an assert call for at least 1-2 LTS versions.

@chancancode chancancode merged commit 974fdaa into main Mar 9, 2024
6 checks passed
@chancancode chancancode deleted the rm-with branch March 9, 2024 02:59
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.

None yet

1 participant