mmdevries / eiows

Other
78 stars 4 forks source link

Use eiows as an optional dependency #22

Closed brako closed 2 months ago

brako commented 2 months ago

Hello,

I am trying to use eiows as an optional dependency on my project. My project is running on Windows and Linux. So I want to use eiows on Linux and switch to native ws on Windows.

But I can't use eiows as an optional dependency, because npm does not know the supported os for the module and will try to install it on Windows anyway.

Is it possible to add explicitly the supported os to the package.json file ?

For example:

{
  "os": ["!win32", "linux", "freebsd", "darwin"]
}

NPM documentation

Thanks!

mmdevries commented 2 months ago

This should now be defined in the 7.1.0 version.

brako commented 2 months ago

I have tested and it is working! thanks