-
-
Notifications
You must be signed in to change notification settings - Fork 123
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
Audit code and reduce build size #386
Comments
@possumbilities @TimidRobot Shall I pick up this issue? |
@Cronus1007 feel free to tackle it, and thankyou! ❤️ |
@possumbilities Unused dependencies which I was able to find are
While running
Screenshots |
@Cronus1007 I could be wrong, but could you provide more info. It seems like several of those are in use in the dependency chain somewhere. I thought The other two I'm not sure on. Is there some kind of output or report indicating that they are indeed not used at all? |
@possumbilities I used the npm dependency i.e. depcheck for checking the unused dependencies. It listed several dependencies and then I went through the codebase checking out the dependencies used which took me to this conclusion. Also when I ran the project after removing these dependencies it went well. Both the serve script and build script were working as previously. I don't think any functionality has been broken due to this. I have a solution for this let me create a branch and commit my changes you could have a look at the build of that branch and if you feel that some functionality has been broken then we can revert everything. Right now I am working on treeshaking in order to reduce the build size as well. |
@Cronus1007 that's a good idea, I'm happy to test the branch |
@possumbilities Sure I am on it. |
I would like to work on it @possumbilities if its possible please assign me |
While building the Chooser as a component, there are several asset size limit warnings, such as the following:
Task
The text was updated successfully, but these errors were encountered: