ember-cli-hot-loader icon indicating copy to clipboard operation
ember-cli-hot-loader copied to clipboard

Consider deprecation or merging with ember-ast-hot-load?

Open rwjblue opened this issue 6 years ago • 2 comments

For those that are unaware, @chancancode and I wrote this test to ensure that it was possible to hot reload templates in Ember and that we don't accidentally break the fundamental ability for this to work in future Ember versions.

Sometime after that @lifeart used the concepts in that test (along with many of the ideas from this addon) to implement https://github.com/lifeart/ember-ast-hot-load.

I'd like to discuss either deprecating this or merging that addon into this one. Thoughts?

rwjblue avatar Dec 10 '18 20:12 rwjblue

There is some questions with possible weak points for ast-hot-load implementation.

1.) Is any stable (public) way to forget components/templates? 2.) Regarding AST transform with MU naming convention it's pretty hard to extract component from templates (it may be local or external property, helper, component, string) (before: foo-bar, after: this.foo, @foo, Boo, FooBar etc). // update - looks like it's fixed 3.) Also, can't figure out how we can deal with nested components matching (currently all components with containing same name part was reloaded). // update - looks like it's fixed 4.) Is it possible to implement "selective" rebuild with new build pipilene? (current implementation waiting for new {appName}.js generate new file with only needed defines. I think we can significantly increase rebuild speed with kinda api to prevent default build behavour and send something like "builThisFile with/without deps" for me. 5.) Personally I think we need to provide glimmer.js support (any pointers to dig?)

I'm pretty sure we can provide kinda ember-cli-hot-loader fallback (if needed) in ast-hot-load version or bump major version ember-cli-hot-loader and backport ast-hot-load findings.

lifeart avatar Dec 10 '18 21:12 lifeart

@rwjblue I'm happy to deprecate this officially. Part of me moving it to this organization was to help sunset it after I got wind @lifeart was rockin' a new addon without all the limitations I ran into building this original prototype.

What is the recommend "process" or documentation to clearly communicate "yup, this is no longer maintained" ?

toranb avatar Dec 10 '18 23:12 toranb