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

Gulp 3.9.1 Issues with Node 10 #2346

Open
c0g1t8 opened this issue Feb 16, 2019 · 4 comments
Open

Gulp 3.9.1 Issues with Node 10 #2346

c0g1t8 opened this issue Feb 16, 2019 · 4 comments

Comments

@c0g1t8
Copy link
Collaborator

c0g1t8 commented Feb 16, 2019

I got everything to build after fixing #2344 and #2345 but the website did not render correctly.
image
There were no build errors. Finally spotted
image indicating that not all the npm dependencies were being installed. Manually restoring resulted in
image

@c0g1t8
Copy link
Collaborator Author

c0g1t8 commented Feb 16, 2019

It turns out this is related to Dependency of Gulp 3.9.1 causes error in Node 10.

I applied the fix as suggested in the link above to resolve. This appeared to have work but did get the following
image

The web pages appear to be rendering correctly but I am unsure about the 15 vulnerabilities reported.

@djonphillips
Copy link

I am experiencing a similar issue with node v10.15.1. The Website does not render correctly and the Visual Studio Task Runner Explorer reports that Gulpfile.js failed to load and the following information was in the output window:
Failed to run "C:\Users\Phillips' PC\source\repos\allReady\AllReadyApp\Web-App\AllReady\Gulpfile.js"...
cmd.exe /c gulp --tasks-simple
gulp --tasks-simple[664]: src\node_contextify.cc:633: Assertion `args[1]->IsString()' failed.
1: 00007FF6C6760EFA v8::internal::GCIdleTimeHandler::GCIdleTimeHandler+4810
2: 00007FF6C673A296 node::MakeCallback+4518
3: 00007FF6C673A34F node::MakeCallback+4703
4: 00007FF6C670D5E8 node::DecodeWrite+12952
5: 00007FF6C6F6ADD2 v8::internal::OptimizingCompileDispatcher::Unblock+60562
6: 00007FF6C6F6BF58 v8::internal::OptimizingCompileDispatcher::Unblock+65048
7: 00007FF6C6F6B28D v8::internal::OptimizingCompileDispatcher::Unblock+61773
8: 00007FF6C6F6B1AB v8::internal::OptimizingCompileDispatcher::Unblock+61547
9: 0000035C0DB5C5C1

@c0g1t8
Copy link
Collaborator Author

c0g1t8 commented Mar 13, 2019

Did some investigation of the AppVeyor build server since this error message did not appear there - see #2351. The build server is using Node 8.15.

Installing Node 8.15 solved this problem - installed https://nodejs.org/download/release/v8.15.0/node-v8.15.0-x64.msi.

In order to ensure that Node 8.15 was being used instead of the Node 10 installed with VS2017 15.9.* the path to the Node 8.15 installation needs to added under the External Web Tools options.

image

@c0g1t8
Copy link
Collaborator Author

c0g1t8 commented Mar 20, 2019

Did some future proofing to support Gulp 4 in PR #2352 which works in both Node 10 and Node 8.15.

Removed knockout-bootstrap - including cause the integration test to fail intermittently. This needs to be researched further.

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

No branches or pull requests

2 participants