Skip to content
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

cryptoTestJS/test gives a bunch of NPM warnings on master #2806

Open
Christewart opened this issue Mar 18, 2021 · 2 comments
Open

cryptoTestJS/test gives a bunch of NPM warnings on master #2806

Christewart opened this issue Mar 18, 2021 · 2 comments
Labels
build dependencies Pull requests that update a dependency file scalajs
Projects
Milestone

Comments

@Christewart
Copy link
Contributor

Christewart commented Mar 18, 2021

When i try to run cryptoTestJS/test on b814005 I get a bunch of npm warnings around a dep in javascript

[error] npm WARN optional SKIPPING OPTIONAL DEPENDENCY: fsevents@1.2.13 (node_modules/watchpack-chokidar2/node_modules/fsevents):
[error] npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for fsevents@1.2.13: wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
[error] npm WARN optional SKIPPING OPTIONAL DEPENDENCY: fsevents@2.3.2 (node_modules/fsevents):
[error] npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for fsevents@2.3.2: wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
[error] npm WARN optional SKIPPING OPTIONAL DEPENDENCY: fsevents@1.2.13 (node_modules/webpack-dev-server/node_modules/fsevents):
[error] npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for fsevents@1.2.13: wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
[info] audited 588 packages in 4.165s
[info] 18 packages are looking for funding
[info] run npm fund for details
[info] found 2 low severity vulnerabilities
[info] run npm audit fix to fix them, or npm audit for details
[info] Fast optimizing /home/chris/dev/bitcoin-s/crypto-test/.js/target/scala-2.13/bitcoin-s-crypto-test-test-fastopt

@Christewart Christewart added build dependencies Pull requests that update a dependency file scalajs labels Mar 18, 2021
@Christewart Christewart added this to the v0.6.0 milestone Mar 18, 2021
@Christewart Christewart added this to Todo in v0.6 via automation Mar 18, 2021
@Christewart
Copy link
Contributor Author

I think this has something to do with our webpack version and a transitive dependency on chokidar

webpack/webpack-dev-server#2528

@Christewart
Copy link
Contributor Author

So I don't believe this can be fixed until webpack-dev-server bumps their dependency on chokidar as the issue above states.

I took a bit of time to see if this problem exists on other scalajs projects, and does

If you cloned laminar and then did

sbt websiteJS/run

and got the same warnings

@Christewart Christewart removed this from Todo in v0.6 Apr 8, 2021
@Christewart Christewart modified the milestones: v0.6.0, v0.7 Apr 8, 2021
@Christewart Christewart modified the milestones: v0.7, 0.8 May 10, 2021
@Christewart Christewart added this to To do in 1.8 Jul 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build dependencies Pull requests that update a dependency file scalajs
Projects
No open projects
1.8
To do
Development

No branches or pull requests

1 participant