Add compatibility for Ember 4
Breaking
Officially drop support for Node < 12
Drop support for Ember <3.24
Chore
Upgrade to Ember 4
- run
ember-cli-update --to 4.4.0andyarn install
Also
- specify node version supported in package.json (>= 12)
- update ember-classic-decorator devDeps to v3.0.1 (v2 was not compatible with Ember 4)
- update ember-data devDeps to 4.4.x (v3 was not compatible with Ember 4)
- run
yarn-deduplicate&&yarn installto clean theyarn.lock - remove
jshintconfig files used in tests (not relevant anymore, was introduced with first versions of Ember, we now useeslintin tests) - lint
- fix some lint errors in test
- fix template lint errors
- ⚠️ temporarily move eslint and template lint error to a warning -> Goal of this PR is to make this repo compatible with Ember 4 so that consumers using Ember 4 can use it ; refactoring whole addon to native class / glimmer may introduce some side effects and it seems not a blocking for Ember 4
Remove object-bin export
Taken from https://github.com/mharris717/ember-drag-drop/pull/194/ (file no longer exists in ./addon/component) should make embroider happy
Build
Remove ember-decorators from devDeps
It was not used by this repo
Motivation
We are working on an old private project which is being migrated to Ember 4. At this moment v0.9.0 is not compatible with Ember 4, so it block us, this PR make ember-drag-drop compatible with Ember 4 and unblock us.
I'm not tackling refactor to glimmer and native class yet on this PR (only switched errors to warnings) because it could brings side effects and I don't have enough knowledge of this repo / our old repo to be sure I will detect them.
If this PR is
- accepted 🎉 I will maybe take some times to work on these refactors and submit some PRs (would be happy to discuss with you @mharris717 ,
ndekeister#0721in Ember Discord, we can maybe think about a roadmap / see if some other people are interested to work on it / contribute to this cool addon) - not accepted 😿 we will probably refactor our drag'n'drop implementation with another packages (but it could be nice to specify in the
README.mdthat addon is not compatible with Ember 4 🙏 )
CI run -> https://github.com/ndekeister-us/ember-drag-drop/pull/1/checks (only embroider-optimized failing, didn't investigated
cc @dgavey
Ok I'll take a look at it.
bump - looking forward to seeing this work!
When is this mr going to be merged ? @tehhowch