Clay: yarn build doesn't work on node v12

[brunofarache]

yarn build doesn't work when using node version 12 or above.

gulp 3 doesn't work well with node 12, see:

https://stackoverflow.com/questions/55921442/how-to-fix-referenceerror-primordials-is-not-defined-in-node

[jbalsas]

Added to https://github.com/liferay/clay/milestone/15 so we can prioritize it :)

[p2kmgcl]

Same thing happens with node 14. Which version of node should we be using? It seems to work with node 10

[jbalsas]

Yeah, @clayui/css depends on gulp@3 for the test site, so I guess that's where the issues is coming from.

https://github.com/liferay/clay/pull/3362 should get us closer to removing the gulp dependency, I think... right @bryceosterhaus?

[bryceosterhaus]

@jbalsas yeah thats correct, there are issues with gulp + newer node versions, but only for clay-css. The other packages should build fine.

@p2kmgcl do you need to build it on node 14 for a specific reason? We can try to prioritize this if its a road block for something.

[jbalsas]

I think they're simply running yarn build from the root to trigger everything, instead of independent packages. And since that does lerna build, it will choke when it gets to build clay-css.

[p2kmgcl]

@bryceosterhaus nope, I don't need to use node 14, it was the version I had installed by default :sweat_smile:
But it's no problem to switch to node 10 :)

[bryceosterhaus]

@p2kmgcl sounds good! We are slowly trending towards getting rid of the gulp dependency, but until its fully removed, we need to build clay-css with node 10

[kresimir-coko]

Is this still a problem @bryceosterhaus after #3362 has been merged?

[bryceosterhaus]

@kresimir-coko yeah this is still an issue for as long as we have gulp used in clay-css. I think we can just keep this open in case others run into this issue

[matuzalemsteles]

I'll close this in favor of #4132, we'll soon start working on simplifying the clay-css build flow to allow us to use the latest versions of Node.js and lessen our dependencies. Thanks guys!

[brunofarache]

👀

[nhpatt]

To whom do those eyes belong? ^ Do you know him? :)

[brunofarache]

Sorry I meant 🎉, I can finally build clay with my favorite node version 😂

[bryceosterhaus]

@brunofarache, I am so excited you have decided to be our number 1 user now! "Elastic built with ClayUI" 😍 😂