Skip to content

[FEATURE default helper manager] Add the default helper manager#20052

Merged
wycats merged 1 commit intoemberjs:masterfrom
Windvis:feature/default-helper-manager
Apr 13, 2022
Merged

[FEATURE default helper manager] Add the default helper manager#20052
wycats merged 1 commit intoemberjs:masterfrom
Windvis:feature/default-helper-manager

Conversation

@Windvis
Copy link
Copy Markdown
Contributor

@Windvis Windvis commented Apr 9, 2022

This updates the Glimmer VM to version 0.84.2 which includes the default helper manager and a feature flag to conditionally enable that feature in Ember.

This PR supersedes #19973.

@Windvis
Copy link
Copy Markdown
Contributor Author

Windvis commented Apr 9, 2022

Draft since more changes are needed in the VM when using functions that are stored on class properties or passed as arguments. The required VM changes have been merged and released in v0.84.2 🎉.

This updates the Glimmer VM to version 0.84.2 which includes the default helper manager and a feature flag to conditionally enable that feature in Ember.
@Windvis Windvis force-pushed the feature/default-helper-manager branch from b081915 to b9436fd Compare April 13, 2022 10:08
Copy link
Copy Markdown
Member

@chancancode chancancode left a comment

Choose a reason for hiding this comment

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

@Windvis do you have anything else you think needs to be done or are you happy with everything? at a glance it seems good to me

@wycats
Copy link
Copy Markdown
Member

wycats commented Apr 13, 2022

@chancancode any reason not to merge this?

@Windvis
Copy link
Copy Markdown
Contributor Author

Windvis commented Apr 13, 2022

@chancancode I think it's good to go. 👍 The RFC mentions some documentation updates but that's part of the guides so nothing needs to be added here I think.

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