-
Notifications
You must be signed in to change notification settings - Fork 77
Update: modernize dependencies #27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Update: modernize dependencies #27
Conversation
Thanks so much for all the work you've put into this. It's a bit hard to review as-is because the change from 2 spaces to 4 made everything part of the diff. Could you maybe back out those changes for now and submit only the stuff that's actually required to upgrade the dependencies? |
Uhh, sorry. Will change spacing part. And try to check if it will be a little clearer to understand the diff. |
In order to keep the package up to date, following dependencies were updated/changed:
Some of the packages, especially backbone and backbone.marionette required breaking changes, so tried to update accordingly.
I've really tried to rewrite backbone logic in React, but essentially it's too much work, I guess an lot of logic will be rewritten to React.
Also, removed marionette modules in favour of ES6 ones.
So, let me know what you think