Skip to content

Reduce package size by excluding unused 'deps' directory #1125

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

shayff
Copy link

@shayff shayff commented Apr 22, 2025

The 'deps' directory in the node-rdkafka package is significantly large (around 98MB), constituting roughly 83% of the total package size. Seems like removing that directory is not affect the functionality.

This pull request proposes removing the 'deps' directory to decrease the package size.

File size breakdown (before this change):

Files:
24K ./bench
88K ./test
16K ./util
24K ./ci
32K ./examples
128K ./lib
98M ./deps
18M ./build
76K ./e2e
264K ./src
118M .

The 'deps' directory in the node-rdkafka package is significantly large (around 98MB), constituting roughly 83% of the total package size. Seems like removing that directory is not affect the functionality.

This pull request proposes removing the 'deps' directory to decrease the package size.

**File size breakdown (before this change):**

Files:
 24K	./bench
 88K	./test
 16K	./util
 24K	./ci
 32K	./examples
128K	./lib
 98M	./deps
 18M	./build
 76K	./e2e
264K	./src
118M	.
@shayff shayff marked this pull request as draft April 22, 2025 10:26
@shayff shayff marked this pull request as ready for review April 22, 2025 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant