-
Notifications
You must be signed in to change notification settings - Fork 988
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
Update tooling #77
base: master
Are you sure you want to change the base?
Update tooling #77
Conversation
Also remove unnecessary polyfills, and build only for the chrome versions needed by AE
Also something I've tried and seems to work well is changing the ExtendScript |
hey thanks so much for all the improvements!! |
Sounds great, let me know if you find anything that you think needs changing! |
I did a build and it didn't work. I'll have to look into it, but has it worked for you? |
Apologies, I didn't properly test the prod build since changing the webpack config, public path needed to be set differently. I've updated it and tested and it seems to be working now :) |
React version is unchanged as its still supported and would require way too much work to be worth it.
If you're wondering why, I'm planning to use this plugin as a basis for an in-house extension at work, and before I get started it'll be nice to have more up-to-date foundations.
EDIT: This will drop support for Node 8 and earlier when developing.