Skip to content

WIP - fix MU style merging#8424

Closed
patricklx wants to merge 1 commit intoember-cli:masterfrom
patricklx:fix-mu-style-merge
Closed

WIP - fix MU style merging#8424
patricklx wants to merge 1 commit intoember-cli:masterfrom
patricklx:fix-mu-style-merge

Conversation

@patricklx
Copy link
Copy Markdown

@patricklx patricklx commented Feb 12, 2019

depends on
#8431

will rebase on it

@patricklx patricklx changed the title fix non css style merging fix MU style merging Feb 12, 2019
@patricklx patricklx force-pushed the fix-mu-style-merge branch 4 times, most recently from 06672c0 to 3b14829 Compare February 13, 2019 15:38
@ppcano
Copy link
Copy Markdown
Contributor

ppcano commented Feb 14, 2019

We have a plan to split this PR in several ones that tackle different MU issues.

@ppcano ppcano mentioned this pull request Feb 14, 2019
options.command = options.command || 'new';

return applyCommand(options.command, projectName, '--skip-npm', '--skip-bower', `--directory=${outputDir}`)
.then(() => isExperimentEnabled('MODULE_UNIFICATION') && getURLFor('canary'))
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This should be fixed by #8432. Thanks for investigating and reporting the issue.

// Here you can enable experimental features on an ember canary build
// e.g. 'with-controller': true
'ember-module-unification': true
EMBER_MODULE_UNIFICATION: true
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This should be fixed in a different PR.

@rwjblue
Copy link
Copy Markdown
Member

rwjblue commented Feb 16, 2019

Needs a rebase, also @ppcano mind doing another review?

@ppcano
Copy link
Copy Markdown
Contributor

ppcano commented Feb 16, 2019

@rwjblue Yes!, Thank you for looking at it.

I am working with @patricklx on these MU changes. We want to merge first #8431, but we will ping you when it is finally ready.

@patricklx patricklx changed the title fix MU style merging WIP - fix MU style merging Feb 16, 2019
@patricklx patricklx force-pushed the fix-mu-style-merge branch 2 times, most recently from 0a6968b to ed475eb Compare February 23, 2019 11:33
@locks
Copy link
Copy Markdown
Contributor

locks commented Apr 9, 2020

MU has been dropped as an initiative. Thanks everyone for your contributions!

@locks locks closed this Apr 9, 2020
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.

4 participants