Skip to content

[WIP] [Feature] helper blueprint for module unification#16492

Closed
RuairiK wants to merge 2 commits intoemberjs:masterfrom
RuairiK:RK/helper-blueprint-pr
Closed

[WIP] [Feature] helper blueprint for module unification#16492
RuairiK wants to merge 2 commits intoemberjs:masterfrom
RuairiK:RK/helper-blueprint-pr

Conversation

@RuairiK
Copy link
Contributor

@RuairiK RuairiK commented Apr 11, 2018

@RuairiK RuairiK force-pushed the RK/helper-blueprint-pr branch 2 times, most recently from f589d30 to b16bb68 Compare April 11, 2018 19:45
@GavinJoyce
Copy link
Member

It looks like we'll have to update ember-cli to skip *-addon blueprints like https://github.com/emberjs/ember.js/tree/master/blueprints/helper-addon, otherwise the unwanted export file in addon/app will be generated:

https://github.com/ember-cli/ember-cli/blob/df70d99caf1e7226126ff9e3a96d03319c161209/lib/tasks/generate-from-blueprint.js#L23

LMK if you'd like to pair on this soon. Otherwise, I'll take this on later today

@GavinJoyce
Copy link
Member

GavinJoyce commented Apr 20, 2018

@RuairiK ^ that's been resolved, perhaps you could rebase?

Also, could you add an in-repo-addon test similar to this? This will ensure that we're not breaking existing functionality.

@RuairiK RuairiK force-pushed the RK/helper-blueprint-pr branch from b16bb68 to 875d19f Compare April 30, 2018 16:10
@GavinJoyce
Copy link
Member

@RuairiK it looks like there are some failing blueprint tests

@GavinJoyce
Copy link
Member

If anyone wants to pick this PR up, please do so

@ppcano
Copy link
Contributor

ppcano commented Nov 15, 2018

#17211 continue and closes this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants