Electron-SDK icon indicating copy to clipboard operation
Electron-SDK copied to clipboard

Published 20 hours ago • verified publisher icon AgoraIO-Extensions

Remove unnesseray dependencies!

Open henrydf opened this issue 1 year ago • 4 comments

Is your feature request related to a problem? Please describe. Why this SDK requires some annoying things such as gulp* and ts-interface-checker in production dependencies?

Describe the solution you'd like Keep production dependencies clear.

Describe alternatives you've considered Or provide some pre-bundled version in dist or somewhere.

Additional context Current dependencies make bundle result mess.

henrydf avatar Apr 12 '24 04:04 henrydf

@henrydf Thanks for your advice! Currently, gulp is necessary, this will download our c++ native modules when you install Electron-SDK. ts-interface-checker is necessary when we check some variables in runtime.

If you have any suggestions please leave a message.

guoxianzhe avatar Apr 12 '24 06:04 guoxianzhe

How about provide a prebuild version of js/ts files. Then I just need to bundle the prebuild result instead of 500+ dependencies.

henrydf avatar Apr 12 '24 06:04 henrydf

@henrydf The main reason for this problem is gulp IMO. Maybe We can remove gulp in the future. I will talk with my team and make it happen in the feature. This gonna be a huge work to do and please wait my further progress.

guoxianzhe avatar Apr 16 '24 02:04 guoxianzhe

Glad to see it.

henrydf avatar Apr 16 '24 02:04 henrydf

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jun 15 '24 13:06 stale[bot]