search

tracked-tools / tracked-toolbox

Helpful autotracking utilities
MIT License
83 stars 13 forks source link

Breaking Change: Remove `@cached` #209

Open elwayman02 opened 3 months ago

elwayman02 commented 3 months ago

Now that it's built into Ember, we should deprecate @cached in this library and release a major version dropping support versions for Ember prior to it being added (4.1, I think?).

NullVoxPopuli commented 3 months ago

if the behavior is the same, the implementation could be a re-export, no need for breaking change.

and then we could deprecate usage of @cached from this library -- and provide a codemod to move to built in

SergeAstapov commented 3 months ago

re-export with deprecation seems feasible to me.

as for codemod, seems like could be similar to https://github.com/ijlee2/ember-codemod-remove-inject-as-service