Massive NPM package about scroll-js

[markcellus]

Just to clarify, this package only has only four dependencies at the time of this writing. These dependencies are necessary to allow the use of ES6 syntax and features while also being backwards compatible (mainly for IE browsers) when building this package manually.

Note that when using the npm package, it is assuming that you are building your own dist file. And even though other nested deps are installed, they are not shipped with your production code, so it shouldn't make a difference of how many packages get installed.

If you are only interested in the smallest JS file possible without having to install npm deps, I suggest you download the dist files directly (as indicated in the Installation instructions on the README).

I am closing this as this is not an issue. However, please feel free to still comment here and I will make time to address any further concerns.

[dominikfiala]

Thanks for your reply! It seems that we differ in NPM philosophy.
I never considered to build dist files of third party packages in Node ecosystem. I always thought that building third-party packages is meant only for development of the package. And I thought that thats the reason for devDependencies.

[markcellus]

@dominikfiala I understand your concerns. The reason the package was built this way was to give developers a way to use their own build processes to ensure the final output file would be as small as possible. However, for the reasons you've stated and out of convenience, I do think it is best at this point to point the npm package at the dist file and remove the unnecessary dependencies to customize the build.

I have done this and added it to version 1.8 (latest version) of this package. Thanks for your contribution.