search

s-KaiNet / spsave

Save files in SharePoint using node.js easily
MIT License
86 stars 22 forks source link

Why do you use node-notifier? #74

Closed atassis closed 3 years ago

atassis commented 3 years ago

node-notifier has nothing to do with uploading data into sharepoint and can be used separately.

s-KaiNet commented 3 years ago

Because it's part of the spsave options

atassis commented 3 years ago

Опции это следствие, а не причина Я бы хотел использовать пакет на сервере, но не хочу тащить лишние зависимости. node-notifier можно спокойно использовать там, где оно необходимо отдельно от вашего пакета

s-KaiNet commented 3 years ago

Набор опций это выбор автора библиотеки. Я не могу уменьшать количество опций только потому, что кто-то не пользуется какими-то из них.

У вас есть несколько путей решения проблемы. Если вы используете webpack, то можно указать node-notifier в externals секции, чтобы он не увеличивал размер бандла. Можно сделать форк spsave и опубликовать свою версию с необходимым функционалом (чтобы убрать нотификации нужно удалить несколько строк). Или смириться с дополнительной "лишней" зависимостью и использовать модуль as-is.

atassis commented 3 years ago

Ок, понял вашу позицию