Support for module unification folder

[hakubo]

No description provided.

[samselikoff]

Thank you!

Could we get a new test added here?

[hakubo]

will do next week,
also - not sure if that works :)
so give me couple of more days to finish that

[hakubo]

@samselikoff turns out much more is needed to support MU layout
I have gone to the point that tests run
everything should be fine

but actual thing that we're trying to do here is not working
could you have a look and point me in the right direction?

since this needs master of ember-cli to run now - probably you won't be merging that soon
but I'd like to test drive it in my side project so I'd be glad to get some help with it

[willviles]

It'd be great to get this addon working with module unification. I believe these are the issues:

treeForStyles isn't used by module unification apps. Instead, everything is merged together via app.trees.src. See ember-cli/ember-cli#7862.

However, at the moment there's a blocking issue with merging trees into app.trees.src, caused by a validation error with Ember CLI ESLint: ember-cli/ember-cli-eslint#56.

[samselikoff]

@willviles are y'all using MU today? It was my understanding that much could change about MU before it gets shipped and I'm a little hesitant to add a bunch of code to ec-tailwind to support an unstable feature...

Happy to do it if it makes sense though

[willviles]

I'm not migrating our production apps to MU yet as compatibility with addons is clearly sketchy. However, I'm testing out a small static marketing site with module unification and reporting problems as I go.

I'll dig a little deeper and find out whether there's a stable path for addons to support MU and what's blocking that from happening 👍

[samselikoff]

Awesome, thank you very much & looking forward to what you find. (Latest thing I saw that made me hesitant was this comment, perhaps Robert + crew have more information.)

[willviles]

Sure - I think that particular comment was relating to a separate RFC titled Module Unification Namespaces and specifically the use of the new {{use}} helper, rather than the accepted and well rationalized Module Unification RFC.

Albeit there's still a fair bit left to be implemented in the quest issue for MU!

[samselikoff]

I see – then yes we should support it. Seems like it's come up a few times

[hakubo]

Thanks @gossi for pushing this foraward